[PATCH] D42100: Fix codegen of stores of vectors with non byte-sized elements.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 16 12:42:09 PST 2018


efriedma added a comment.

I don't see a change to DAGTypeLegalizer::SplitVecOp_STORE?  (I think you're currently miscompiling the `<8 x i31>` testcase.)

We're generating pretty terrible code for some of the x86 testcases, but you obviously don't need to fix that here.

For the assertion failure, it looks like you're uncovering some existing bug in computeKnownBits; I can't think of any way your changes could cause that issue.  Maybe some bug in the bitcast handling code?


https://reviews.llvm.org/D42100





More information about the llvm-commits mailing list