[PATCH] D21146: Fix BSS global handling in AsmPrinter
Rafael Espíndola via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 14 03:21:23 PDT 2016
Lgtm. Do you have commit access?
BTW, I don't think there is any value in using a .lcomm or a .local/.comm
pair instead of . section. A followup patch changing that also lgtm :-)
On Jun 8, 2016 2:15 PM, "Nirav Dave" <niravd at google.com> wrote:
niravd created this revision.
niravd added reviewers: echristo, rafael.
niravd added a subscriber: llvm-commits.
Herald added a subscriber: mehdi_amini.
Change EmitGlobalVariable to check final assembler section is in BSS
before using .lcomm/.comm directive. This prevents globals from being
put into .bss erroneously when -data-sections is used.
This fixes PR26570.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-commits