[PATCH] D143171: [NFC] PHITransAddr refactoring

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 2 06:09:07 PST 2023


mkazantsev requested changes to this revision.
mkazantsev added a comment.
This revision now requires changes to proceed.

I'm generally fine with the whole thing, but I'm against merging several unrelated change sets into one patch. They should be at least somehow related by thematics.

Be merciful to the reviewers, such a salad makes eyes bleed. :(

Please separate them, like:

- A patch that changes API of translateValue;
- A patch that changes loops with counters into standard;
- A patch that removes isSafeToSpeculativelyExecute

I'm sure that separately none of them will cause any trouble. I'm just asking to make it edible for review.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143171



More information about the llvm-commits mailing list