diff options
| author | Hans Wennborg <hans@hanshq.net> | 2014-10-16 20:52:46 +0000 |
|---|---|---|
| committer | Hans Wennborg <hans@hanshq.net> | 2014-10-16 20:52:46 +0000 |
| commit | 899ded9cdf53b3d84c8d0e771851cc256296bfd2 (patch) | |
| tree | 05e0f0f099bab9d80d79172a08299e85c89d306f /llvm/lib/Support/CommandLine.cpp | |
| parent | 933bead97d1cd46e29d347329442a3c6f7259ce7 (diff) | |
| download | llvm-899ded9cdf53b3d84c8d0e771851cc256296bfd2.zip llvm-899ded9cdf53b3d84c8d0e771851cc256296bfd2.tar.gz llvm-899ded9cdf53b3d84c8d0e771851cc256296bfd2.tar.bz2 | |
MS Compat: mark globals emitted in read-only sections const
They cannot be written to, so marking them const makes sense and may improve
optimisation.
As a side-effect, SectionInfos has to be moved from Sema to ASTContext.
It also fixes this problem, that occurs when compiling ATL:
warning LNK4254: section 'ATL' (C0000040) merged into '.rdata' (40000040) with different attributes
The ATL headers are putting variables in a special section that's marked
read-only. However, Clang currently can't model that read-onlyness in the IR.
But, by making the variables const, the section does become read-only, and
the linker warning is avoided.
Differential Revision: http://reviews.llvm.org/D5812
llvm-svn: 219960
Diffstat (limited to 'llvm/lib/Support/CommandLine.cpp')
0 files changed, 0 insertions, 0 deletions