[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