From 7c4fdfe5f4d6ca9173adb1c375311dc929a4b81f Mon Sep 17 00:00:00 2001
From: Lang Hames <lhames@gmail.com>
Date: Wed, 5 Sep 2018 20:57:41 +0000
Subject: [PATCH] Remove some unnecessary constructor arguments.

ExecutionSession defaults to creating a new shared pool if none is provided,
so explicitly passing one in is unnecessary.

llvm-svn: 341494
---
 .../ExecutionEngine/Orc/RTDyldObjectLinkingLayer2Test.cpp | 8 ++++----
 .../ExecutionEngine/Orc/RTDyldObjectLinkingLayerTest.cpp  | 8 ++++----
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/unittests/ExecutionEngine/Orc/RTDyldObjectLinkingLayer2Test.cpp b/unittests/ExecutionEngine/Orc/RTDyldObjectLinkingLayer2Test.cpp
index d6f48c992ae..30cd923b54a 100644
--- a/unittests/ExecutionEngine/Orc/RTDyldObjectLinkingLayer2Test.cpp
+++ b/unittests/ExecutionEngine/Orc/RTDyldObjectLinkingLayer2Test.cpp
@@ -67,7 +67,7 @@ static bool testSetProcessAllSections(std::unique_ptr<MemoryBuffer> Obj,
   bool DebugSectionSeen = false;
   auto MM = std::make_shared<MemoryManagerWrapper>(DebugSectionSeen);
 
-  ExecutionSession ES(std::make_shared<SymbolStringPool>());
+  ExecutionSession ES;
   auto &JD = ES.createJITDylib("main");
   auto Foo = ES.getSymbolStringPool().intern("foo");
 
@@ -261,7 +261,7 @@ TEST(RTDyldObjectLinkingLayer2Test, NoDuplicateFinalization) {
     return;
 
   LLVMContext Context;
-  ExecutionSession ES(std::make_shared<SymbolStringPool>());
+  ExecutionSession ES;
   auto &JD = ES.createJITDylib("main");
 
   auto Foo = ES.getSymbolStringPool().intern("foo");
@@ -338,7 +338,7 @@ TEST(RTDyldObjectLinkingLayer2Test, NoPrematureAllocation) {
   if (!TM)
     return;
 
-  ExecutionSession ES(std::make_shared<SymbolStringPool>());
+  ExecutionSession ES;
   auto &JD = ES.createJITDylib("main");
 
   auto Foo = ES.getSymbolStringPool().intern("foo");
@@ -392,7 +392,7 @@ TEST(RTDyldObjectLinkingLayer2Test, NoPrematureAllocation) {
 }
 
 TEST(RTDyldObjectLinkingLayer2Test, TestNotifyLoadedSignature) {
-  ExecutionSession ES(std::make_shared<SymbolStringPool>());
+  ExecutionSession ES;
   RTDyldObjectLinkingLayer2 ObjLayer(
       ES,
       [](VModuleKey) -> std::shared_ptr<RuntimeDyld::MemoryManager> {
diff --git a/unittests/ExecutionEngine/Orc/RTDyldObjectLinkingLayerTest.cpp b/unittests/ExecutionEngine/Orc/RTDyldObjectLinkingLayerTest.cpp
index 94b771f2640..62c6b7dfa31 100644
--- a/unittests/ExecutionEngine/Orc/RTDyldObjectLinkingLayerTest.cpp
+++ b/unittests/ExecutionEngine/Orc/RTDyldObjectLinkingLayerTest.cpp
@@ -67,7 +67,7 @@ TEST(RTDyldObjectLinkingLayerTest, TestSetProcessAllSections) {
   bool DebugSectionSeen = false;
   auto MM = std::make_shared<MemoryManagerWrapper>(DebugSectionSeen);
 
-  ExecutionSession ES(std::make_shared<SymbolStringPool>());
+  ExecutionSession ES;
 
   RTDyldObjectLinkingLayer ObjLayer(ES, [&MM](VModuleKey) {
     return RTDyldObjectLinkingLayer::Resources{
@@ -123,7 +123,7 @@ TEST_F(RTDyldObjectLinkingLayerExecutionTest, NoDuplicateFinalization) {
   if (!SupportsJIT)
     return;
 
-  ExecutionSession ES(std::make_shared<SymbolStringPool>());
+  ExecutionSession ES;
 
   auto MM = std::make_shared<SectionMemoryManagerWrapper>();
 
@@ -207,7 +207,7 @@ TEST_F(RTDyldObjectLinkingLayerExecutionTest, NoPrematureAllocation) {
   if (!SupportsJIT)
     return;
 
-  ExecutionSession ES(std::make_shared<SymbolStringPool>());
+  ExecutionSession ES;
 
   auto MM = std::make_shared<SectionMemoryManagerWrapper>();
 
@@ -268,7 +268,7 @@ TEST_F(RTDyldObjectLinkingLayerExecutionTest, NoPrematureAllocation) {
 }
 
 TEST_F(RTDyldObjectLinkingLayerExecutionTest, TestNotifyLoadedSignature) {
-  ExecutionSession ES(std::make_shared<SymbolStringPool>());
+  ExecutionSession ES;
   RTDyldObjectLinkingLayer ObjLayer(
       ES,
       [](VModuleKey) {