[PATCH] Replace the result usages while legalizing cmpxchg

Logan Chien tzuhsiang.chien at gmail.com
Mon Jul 21 10:23:29 PDT 2014


Hi,

I have revised the test cases.  Please have a look.  Thanks.

Sincerely,
Logan


On Fri, Jul 18, 2014 at 2:32 AM, Tim Northover <t.p.northover at gmail.com>
wrote:

> Hi Logan,
>
> Thanks for fixing this. The change itself looks fine, but the tests
> seem a bit weak (basically just "does this compile" if I'm reading
> them correctly). Any chance you could check the actual code generated
> too?
>
> Cheers.
>
> Tim.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140722/a0667a68/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Replace-the-result-usages-while-legalizing-cmpxchg.patch
Type: text/x-patch
Size: 7258 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140722/a0667a68/attachment.bin>


More information about the llvm-commits mailing list