<div dir="ltr">LGTM.<div><br></div><div>-Eli</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Sep 10, 2013 at 5:10 PM, Matt Arsenault <span dir="ltr"><<a href="mailto:Matthew.Arsenault@amd.com" target="_blank">Matthew.Arsenault@amd.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">  Move address space check for memset_pattern16<br>
<br>
<br>
  Most of the address space tests are copies of other tests. They aren't supposed to do much different other than use the different integer sizes and preserve the address spaces.<br>
<div class="im"><br>
<a href="http://llvm-reviews.chandlerc.com/D1462" target="_blank">http://llvm-reviews.chandlerc.com/D1462</a><br>
<br>
CHANGE SINCE LAST DIFF<br>
</div>  <a href="http://llvm-reviews.chandlerc.com/D1462?vs=3780&id=4183#toc" target="_blank">http://llvm-reviews.chandlerc.com/D1462?vs=3780&id=4183#toc</a><br>
<div class="HOEnZb"><div class="h5"><br>
Files:<br>
  lib/Transforms/Scalar/LoopIdiomRecognize.cpp<br>
  test/Transforms/IndVarSimplify/lftr-address-space-pointers.ll<br>
  test/Transforms/LoopIdiom/basic-address-space.ll<br>
  test/Transforms/LoopStrengthReduce/addrec-gep-address-space.ll<br>
  test/Transforms/LoopStrengthReduce/address-space-loop.ll<br>
  test/Transforms/LoopStrengthReduce/uglygep-address-space.ll<br>
</div></div></blockquote></div><br></div>