[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