Insert MCSectionData into the section list directly in getOrCreateSectionData.

Removing the logic from the constructor will make it easier to merge
MCSectionData and MCSection.

llvm-svn: 238155

2 files changed, 6 insertions, 8 deletions

diff --git a/llvm/include/llvm/MC/MCAssembler.h b/llvm/include/llvm/MC/MCAssembler.h
index ee7583e0e82..59f8e0ad41b 100644
--- a/llvm/include/llvm/MC/MCAssembler.h
+++ b/llvm/include/llvm/MC/MCAssembler.h
@@ -566,7 +566,7 @@ private:
 public:
   // Only for use as sentinel.
   MCSectionData();
-  MCSectionData(MCSection &Section, MCAssembler *A = nullptr);
+  MCSectionData(MCSection &Section);
 
   MCSection &getSection() const { return *Section; }
 
@@ -971,8 +971,10 @@ public:
 
     if (Created)
       *Created = !Entry;
-    if (!Entry)
-      Entry = new MCSectionData(Section, this);
+    if (!Entry) {
+      Entry = new MCSectionData(Section);
+      Sections.push_back(Entry);
+    }
 
     return *Entry;
   }
diff --git a/llvm/lib/MC/MCAssembler.cpp b/llvm/lib/MC/MCAssembler.cpp
index d7712f7fc98..2fa023ee120 100644
--- a/llvm/lib/MC/MCAssembler.cpp
+++ b/llvm/lib/MC/MCAssembler.cpp
@@ -292,11 +292,7 @@ MCEncodedFragmentWithFixups::~MCEncodedFragmentWithFixups() {
 
 MCSectionData::MCSectionData() : Section(nullptr) {}
 
-MCSectionData::MCSectionData(MCSection &Section, MCAssembler *A)
-    : Section(&Section) {
-  if (A)
-    A->getSectionList().push_back(this);
-}
+MCSectionData::MCSectionData(MCSection &Section) : Section(&Section) {}
 
 MCSectionData::iterator
 MCSectionData::getSubsectionInsertionPoint(unsigned Subsection) {