<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Wed, Feb 28, 2018 at 1:02 PM Alexey Bataev via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">ABataev added a comment.<br>
<br>
In <a href="https://reviews.llvm.org/D43627#1022609" rel="noreferrer" target="_blank">https://reviews.llvm.org/D43627#1022609</a>, @echristo wrote:<br>
<br>
> Using sections as references is, in general, a good idea when we can so I'd rather not tie that to whether or not we emit certain sections. If you don't mind reverting that aspect of the change I'd appreciate it and we can get that in via some other preference or a strict requirement via the asm printer as we're planning on doing for the rest of the nvptx section emission.<br>
<br>
<br>
These sections have labels inside of them + they are not supported by PTX format. Ok, I will revert the patch.<br><br></blockquote><div><br></div><div>Sure. Adding back everything else is fine to commit btw. We can worry about when to use section relocations rather than label relocations in a different patch though.</div><div><br></div><div>-eric </div></div></div>