[PATCH] D138452: [ConstraintElim] Queue facts and checks directly.

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 5 07:38:23 PST 2022


nikic accepted this revision.
nikic added a comment.
This revision is now accepted and ready to land.

LGTM. A question I had is why we use this kind of upfront collection + sort approach at all. Wouldn't it be possible to do essentially the same in a single walk over the dominator tree?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138452



More information about the llvm-commits mailing list