[PATCH] D20905: Only attempt to detect AVG if SSE2 is available
Sanjay Patel via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 2 08:06:27 PDT 2016
spatel accepted this revision.
spatel added a comment.
This revision is now accepted and ready to land.
LGTM. Thanks!
I was told sometime a while back that we should prefer to give regression test files a meaningful name rather than 'pr###'. Then, you make the function name be '@PR###' rather than '@fn1' or add a comment line to explain further. In this case, a better test file name might be something like 'no-sse2-avg.ll'?
http://reviews.llvm.org/D20905
More information about the llvm-commits
mailing list