<html>

  <head>

    <meta content="text/html; charset=windows-1252"

      http-equiv="Content-Type">

  </head>

  <body bgcolor="#FFFFFF" text="#000000">

    <font face="Calibri">Seems reasonable to me. Thanks for cleaning

      this up!</font><br>

    <br>

    <div class="moz-cite-prefix">On 7/6/2016 6:39 PM, Justin Bogner

      wrote:<br>

    </div>

    <blockquote cite="mid:m2oa6a5r3r.fsf@chronotis.apple.com"

      type="cite">

      <pre wrap="">Justin Lebar <a class="moz-txt-link-rfc2396E" href="mailto:jlebar@google.com"><jlebar@google.com></a> writes:

</pre>

      <blockquote type="cite">

        <blockquote type="cite">

          <pre wrap="">I'll wait on the ones that drop the ptx intrinsics and builtins

until I hear back about the ones that don't seem to have an nvvm

equivalent.

</pre>

        </blockquote>

        <pre wrap="">

What's the outstanding question, exactly?  It seems to me we should

just rename them.

</pre>

      </blockquote>

      <pre wrap="">

Ah, I misunderstood before.

This ended up being a bit more work on the LLVM side, but the attached

patch replaces all of the ptx.read intrinsics with nvvm.read.ptx.sreg

variants, cleans up the duplicate nvvm variants that leaves around, and

renames ptx.bar.sync to nvvm.bar.sync.

The clang patch just updates the names and doesn't change anything with

this version.

WDYT?

</pre>

    </blockquote>

    <br>

<DIV>

<HR>

</DIV>

<DIV>This email message is for the sole use of the intended recipient(s) and may 

contain confidential information.  Any unauthorized review, use, disclosure 

or distribution is prohibited.  If you are not the intended recipient, 

please contact the sender by reply email and destroy all copies of the original 

message. </DIV>

<DIV>

<HR>

</DIV>

</body>

</html>