[PATCH] D10838: Fix invalid shufflevector operands
Simon Pilgrim
llvm-dev at redking.me.uk
Mon Jul 27 03:51:01 PDT 2015
RKSimon added a comment.
In http://reviews.llvm.org/D10838#212425, @mpflanzer wrote:
> Hi Simon, is this test case enough? Clang will fail with an exit code other than 0 if wrong code is generated. The test passes with my changes. Or should I add a check to the test that something like `<2 x i64> <i64 0, i64 undef>` is not generated?
Yes please add a FileCheck stage to the test to give some insight as to what you expect the result to be.
A comment would be useful as well (reference the PR# and a brief explanation of the bug).
http://reviews.llvm.org/D10838
More information about the cfe-commits
mailing list