[PATCH] D35917: [mips] Implement -muninit-const-in-rodata

Eli Friedman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 3 11:51:14 PDT 2017


efriedma added inline comments.


================
Comment at: cfe/trunk/lib/CodeGen/TargetInfo.cpp:6666
+          GVar->setLinkage(llvm::GlobalValue::ExternalLinkage);
+          GVar->setSection("rodata");
+        }
----------------
Also, this change is clearly unacceptable; among other things, messing with the linkage of static local variables is likely to lead to link errors.


Repository:
  rL LLVM

https://reviews.llvm.org/D35917





More information about the cfe-commits mailing list