<p dir="ltr">Lgtm. Do you have commit access?</p>
<p dir="ltr">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 :-)</p>
<p dir="ltr">Cheers,<br>
Rafael</p>
<div class="gmail_quote">On Jun 8, 2016 2:15 PM, "Nirav Dave" <<a href="mailto:niravd@google.com">niravd@google.com</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">niravd created this revision.<br>
niravd added reviewers: echristo, rafael.<br>
niravd added a subscriber: llvm-commits.<br>
Herald added a subscriber: mehdi_amini.<br>
<br>
Change EmitGlobalVariable to check final assembler section is in BSS<br>
before using .lcomm/.comm directive. This prevents globals from being<br>
put into .bss erroneously when -data-sections is used.<br>
<br>
This fixes PR26570.<br>
<br>
<a href="http://reviews.llvm.org/D21146" rel="noreferrer" target="_blank">http://reviews.llvm.org/D21146</a><br>
<br>
Files:<br>
  lib/CodeGen/AsmPrinter/AsmPrinter.cpp<br>
  test/CodeGen/X86/global-sections.ll<br>
<br>
</blockquote></div>