<div dir="ltr"><div>I didn't find any evidence that libdevice has anything related to i128's. And seems, in general, i128 numbers are not very popular in CUDA programming :)</div><div>Thanks for the advice!</div><div><br></div><div><br><div class="gmail_quote"><div dir="ltr">On Wed, Jul 12, 2017 at 4:10 PM Hal Finkel <<a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div bgcolor="#FFFFFF" text="#000000">
    <p><br>
    </p>
    <div class="m_6426332255185695417moz-cite-prefix">On 07/11/2017 03:21 PM, Denys Zariaiev
      via cfe-dev wrote:<br>
    </div>
    <blockquote type="cite">
      
      <div dir="ltr">Hello,
        <div><br>
        </div>
        <div>I'm curious, is it safe to change an alignment i128 type
          for NVPTX target? I made a <a href="https://reviews.llvm.org/D34555" target="_blank">patch to the LLVM
            backend</a> and added `-i128:128` into data layout.</div>
        <div><br>
        </div>
        <div>But after it became that LLVM and Clang layouts are
          different and error is thrown at Clang side:</div>
        <div>> backend data layout
          'e-i64:64-i128:128-v16:16-v32:32-n16:32:64' does not match
          expected target description 'e-i64:64-v16:16-v32:32-n16:32:64'</div>
        <div><br>
        </div>
        <div>Could you please help with advice, can I just make another
          patch for Clang to modify the layout? Won't it break any
          backward compatibility?</div>
      </div>
    </blockquote>
    <br></div><div bgcolor="#FFFFFF" text="#000000">
    There could be some backward compatibility issues with linking new
    IR files with old IR files (because you're increasing the alignment
    requirements for i128), but often no more than other ISAs when new
    SIMD instructions are added, etc. For the NVPTX backend, this might
    be a concern if it affects anything in libdevice (the bitcode
    library that is shipped with CUDA in bitcode form), otherwise, I
    don't think it is a concern. In general, however, I think you should
    just update Clang.<br>
    <br>
     -Hal<br>
    <br>
    <blockquote type="cite"></blockquote></div><div bgcolor="#FFFFFF" text="#000000"><blockquote type="cite">
      <div dir="ltr">
        <div><br>
        </div>
        <div>Thanks a lot,</div>
        <div>Denys Zariaiev</div>
      </div>
      <br>
      <fieldset class="m_6426332255185695417mimeAttachmentHeader"></fieldset>
      <br>
      </blockquote></div><div bgcolor="#FFFFFF" text="#000000"><blockquote type="cite"><pre>_______________________________________________
cfe-dev mailing list
<a class="m_6426332255185695417moz-txt-link-abbreviated" href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a>
<a class="m_6426332255185695417moz-txt-link-freetext" href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a>
</pre>
    </blockquote></div><div bgcolor="#FFFFFF" text="#000000">
    <br>
    <pre class="m_6426332255185695417moz-signature" cols="72">-- 
Hal Finkel
Lead, Compiler Technology and Programming Languages
Leadership Computing Facility
Argonne National Laboratory</pre>
  </div></blockquote></div></div></div>