[PATCH] D136827: [JT][CT] Preserve exisiting BPI/BFI during JumpThreading

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 10 00:38:40 PST 2023


mkazantsev requested changes to this revision.
mkazantsev added a comment.
This revision now requires changes to proceed.
Herald added a subscriber: StephenFan.

PDT seems completely irrelevant to what you are doing in this patch. If you have plans about it, I propose to split off PDT-related part into a separate patch, and make this one dedicated to BPI/BFI only.



================
Comment at: llvm/lib/Transforms/Scalar/JumpThreading.cpp:2406
+  auto *BFI = getOrCreateBFI();
+  auto *BPI = getOrCreateBPI(BFI != nullptr);
+
----------------
`/*name boolean parameter*/`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136827



More information about the llvm-commits mailing list