[PATCH] D130895: Make VL choosing for a splat constant based on its users

Tue Aug 2 05:43:43 PDT 2022

asi-sc added a comment.

In D130895#3691031 <https://reviews.llvm.org/D130895#3691031>, @craig.topper wrote:

> What if the user is the merge operand of a tail undisturbed instruction. The VL of the using instruction would not apply to it then.

Thanks for catching that! I've looked at how to fix this problem and while we can easily check the policy, it seems not possible for now to figure out whether intrinsic has merge operand. We can extend intrinsics markup explicitly saying the position of the merge operand similar to VLOperand and ScalarOperand. What do you think? The fix should not be hard if the position of merge operands is known.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D130895/new/

https://reviews.llvm.org/D130895