<div dir="ltr">Here is a new patch. Please review it.<br><div class="gmail_quote"><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, May 31, 2013 at 7:55 AM, Akira Hatanaka <span dir="ltr"><<a href="mailto:ahatanak@gmail.com" target="_blank">ahatanak@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Can you add the command line and check strings to file test/CodeGen/Mips/atomic.ll instead of creating a new test atomic-eb.ll (see test/CodeGen/Mips/unalignedload.ll)?</div>

<div><br></div><div>The changes you made in MIpsISelLowering.cpp look fine to me.</div>
<div><br></div><div class="gmail_extra"><div class="gmail_quote"><div>On Wed, May 29, 2013 at 8:49 PM, Jyun-Yan You <span dir="ltr"><<a href="mailto:jyyou@cs.nctu.edu.tw" target="_blank">jyyou@cs.nctu.edu.tw</a>></span> wrote:<br>


</div><blockquote class="gmail_quote" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div>
<div dir="ltr">
The MIPS codegen of atomic operations only considers little-endian. This patch adds big-endian support and a test case.<br></div>
<br></div>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div></div>
</blockquote></div><br></div>
</div></div></div><br></div>