[PATCH] D106286: [RISCV] Add a test showing an incorrect vsetvli insertion

[Fraser Cormack via Phabricator via llvm-commits]

frasercrmck added a comment.

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

> Why is preserving vl wrong here? Isn't the vl 4 regardless of which it came from?
>
> I believe the pass is preserving it because it believes that AVL=4 will produce the same vl as long as sew/lmul is constant which it should be for sew=64/lmul=2 vs sew=32/lmul=1

You're right, I must have reduced the test case too far at one point and stopped thinking properly. I'll go back to the original failing case.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106286