Remove apparently unnecessary filter for CMO serializing references to globalinit once variables

jckarter · jckarter · commit 66e2ce691a6b · 2020-08-26T11:38:16.000-07:00
diff --git a/lib/SILOptimizer/IPO/CrossModuleSerializationSetup.cpp b/lib/SILOptimizer/IPO/CrossModuleSerializationSetup.cpp
@@ -337,9 +337,6 @@ bool CrossModuleSerializationSetup::canSerialize(SILInstruction *inst,
         });
     return canUse;
   }
-  if (auto *GAI = dyn_cast<GlobalAddrInst>(inst)) {
-    return !GAI->getReferencedGlobal()->getName().startswith("globalinit_");
-  }
   if (auto *MI = dyn_cast<MethodInst>(inst)) {
     return !MI->getMember().isForeign;
   }

Original file line number	Diff line number	Diff line change
`@@ -337,9 +337,6 @@ bool CrossModuleSerializationSetup::canSerialize(SILInstruction *inst,`
`337`	`337`	`});`
`338`	`338`	`return canUse;`
`339`	`339`	`}`
`340`		`- if (auto *GAI = dyn_cast<GlobalAddrInst>(inst)) {`
`341`		`- return !GAI->getReferencedGlobal()->getName().startswith("globalinit_");`
`342`		`- }`
`343`	`340`	`if (auto *MI = dyn_cast<MethodInst>(inst)) {`
`344`	`341`	`return !MI->getMember().isForeign;`
`345`	`342`	`}`