[PATCH] D136912: [AArch64] precommit tests for D136623, NFC
Sanjay Patel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 8 05:00:01 PST 2022
spatel resigned from this revision.
spatel added inline comments.
================
Comment at: llvm/test/Transforms/InstCombine/add4.ll:151
+; CHECK-NEXT: [[A0:%.*]] = add i16 [[X:%.*]], -1
+; CHECK-NEXT: [[M:%.*]] = shl i16 [[X]], 2
+; CHECK-NEXT: [[A1:%.*]] = add i16 [[M]], [[A0]]
----------------
As I wrote previously, this is not a good test of "x*4". It is a test of "x<<2" as shown by the CHECKs.
Unit-tests should avoid other transforms, so they do not break or become meaningless from independent changes.
Hopefully, other reviewers can explain this better. Since I can't seem to get this point across, I'll resign.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D136912/new/
https://reviews.llvm.org/D136912
More information about the llvm-commits
mailing list