[llvm] r257290 - OrcJITTests//ObjectLinkingLayerTest.cpp: Appease msc18's C2327. It seems definition of nested class would confuse the context.

NAKAMURA Takumi via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 10 07:56:50 PST 2016


Author: chapuni
Date: Sun Jan 10 09:56:49 2016
New Revision: 257290

URL: http://llvm.org/viewvc/llvm-project?rev=257290&view=rev
Log:
OrcJITTests//ObjectLinkingLayerTest.cpp: Appease msc18's C2327. It seems definition of nested class would confuse the context.

  llvm\unittests\ExecutionEngine\Orc\ObjectLinkingLayerTest.cpp(115) : error C2327: 'llvm::OrcExecutionTest::TM' : is not a type name, static, or enumerator
  llvm\unittests\ExecutionEngine\Orc\ObjectLinkingLayerTest.cpp(115) : error C2065: 'TM' : undeclared identifier

FYI, "this->TM" was valid even before moving class SectionMemoryManagerWrapper.

Modified:
    llvm/trunk/unittests/ExecutionEngine/Orc/ObjectLinkingLayerTest.cpp

Modified: llvm/trunk/unittests/ExecutionEngine/Orc/ObjectLinkingLayerTest.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/unittests/ExecutionEngine/Orc/ObjectLinkingLayerTest.cpp?rev=257290&r1=257289&r2=257290&view=diff
==============================================================================
--- llvm/trunk/unittests/ExecutionEngine/Orc/ObjectLinkingLayerTest.cpp (original)
+++ llvm/trunk/unittests/ExecutionEngine/Orc/ObjectLinkingLayerTest.cpp Sun Jan 10 09:56:49 2016
@@ -26,6 +26,15 @@ class ObjectLinkingLayerExecutionTest :
                                         public OrcExecutionTest {
 };
 
+class SectionMemoryManagerWrapper : public SectionMemoryManager {
+public:
+  int FinalizationCount = 0;
+  bool finalizeMemory(std::string *ErrMsg = 0) override {
+    ++FinalizationCount;
+    return SectionMemoryManager::finalizeMemory(ErrMsg);
+  }
+};
+
 TEST(ObjectLinkingLayerTest, TestSetProcessAllSections) {
 
   class SectionMemoryManagerWrapper : public SectionMemoryManager {
@@ -102,15 +111,6 @@ TEST_F(ObjectLinkingLayerExecutionTest,
   if (!TM)
     return;
 
-  class SectionMemoryManagerWrapper : public SectionMemoryManager {
-  public:
-    int FinalizationCount = 0;
-    bool finalizeMemory(std::string *ErrMsg = 0) override {
-      ++FinalizationCount;
-      return SectionMemoryManager::finalizeMemory(ErrMsg);
-    }
-  };
-
   ObjectLinkingLayer<> ObjLayer;
   SimpleCompiler Compile(*TM);
 




More information about the llvm-commits mailing list