[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