[PATCH] D35917: [mips] Implement -muninit-const-in-rodata
Simon Dardis via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 3 12:05:58 PDT 2017
sdardis added inline comments.
================
Comment at: cfe/trunk/lib/CodeGen/TargetInfo.cpp:6666
+ GVar->setLinkage(llvm::GlobalValue::ExternalLinkage);
+ GVar->setSection("rodata");
+ }
----------------
efriedma wrote:
> Also, this change is clearly unacceptable; among other things, messing with the linkage of static local variables is likely to lead to link errors.
I will fix that.
Repository:
rL LLVM
https://reviews.llvm.org/D35917
More information about the cfe-commits
mailing list