<div dir="ltr"><div>Oh, sure, I know you need it I just wanted to separate "can we use a section relocation" from "should we emit these sections". The latter can be handled the same way we're planning to handle the rest of the conditional emission.</div><div><br></div><div>-eric</div><div><br><div class="gmail_quote"><div dir="ltr">On Wed, Feb 28, 2018 at 1:08 PM Alexey Bataev <<a href="mailto:a.bataev@outlook.com" target="_blank">a.bataev@outlook.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div>
    <pre class="m_9011619986163471737m_-259003881204744291moz-signature" cols="72">Unfortunately, I can't commit everything else without this patch. It will definitely break the compilation of ptx files.

-------------
Best regards,
Alexey Bataev</pre>
    <div class="m_9011619986163471737m_-259003881204744291moz-cite-prefix">28.02.2018 16:04, Eric Christopher
      пишет:<br>
    </div></div><div>
    <blockquote type="cite">
      
      <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" target="_blank">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://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Freviews.llvm.org%2FD43627%231022609&data=02%7C01%7C%7C7bbf5fc8944e49fc542008d57eeeed99%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636554487011067680&sdata=kd5OVnYezYu1jsqgji8K03%2FYJYvIpDWZiRCmL2cBiwQ%3D&reserved=0" 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>
    </blockquote>
    <br>
  </div></blockquote></div></div></div>