[PATCH] D128870: [BOLT] Don't apply ICP to instructions with unknown control flow
Amir Ayupov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 25 18:00:17 PDT 2022
Amir added a comment.
In D128870#3677746 <https://reviews.llvm.org/D128870#3677746>, @maksfb wrote:
> Filtering out functions with `hasUnknownControlFlow()` property in `shouldOptimize()` sounds more straightforward. But if you feel confident that some call sites could still be optimized, go for it.
Good idea to filter on a function basis.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D128870/new/
https://reviews.llvm.org/D128870
More information about the llvm-commits
mailing list