[PATCH] D26031: [TargetLowering] Fix undef vector element issue with true/false result handling

Hal Finkel via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 27 11:49:29 PDT 2016


hfinkel added a comment.

My inclination is to leave the code as is, and update the comment to match. If we have all undefs, then we return false (presumably, we have better ways to fold an all-undef operand), but a few undefs are fine.

On the other hand, do we generally produce better code if we don't handle undefs at all?


Repository:
  rL LLVM

https://reviews.llvm.org/D26031





More information about the llvm-commits mailing list