[PATCH] D134229: [RISCV] Make computeIncomingVLVTYPE more conservative when merging predecessor state.
Philip Reames via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 19 15:03:30 PDT 2022
reames accepted this revision.
reames added a comment.
This revision is now accepted and ready to land.
LGTM
For context, Craig and I debugged this earlier today. We had an infinite loop issue which came down to the fact we allowed two waves of updates to propagate through a loop, and the merge points allowed us to toggle back and forth between states instead of falling to the bottom value of the lattice. No known small reproducer at this time unfortunately.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D134229/new/
https://reviews.llvm.org/D134229
More information about the llvm-commits
mailing list