[PATCH] D106593: [WIP][SimplifyCFG] Replace FoldTwoEntryPHINode i1 hack with a different hack.
Roman Lebedev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 22 13:55:27 PDT 2021
lebedev.ri added a comment.
I like this, it's far less fragile,
but as https://bugs.llvm.org/show_bug.cgi?id=51149 reports,
there are some bad performance concerns with `FoldTwoEntryPHINode`,
and in fact i just tuned/fixed heuristic in rG7ef6f019090f3979fa345105b9ac95ac589c6cf9 <https://reviews.llvm.org/rG7ef6f019090f3979fa345105b9ac95ac589c6cf9>
to workaround them.
Presumably `FoldTwoEntryPHINode` is missing some other profitability check.
Not sure about plain build, but we could at least try to use branch weights.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D106593/new/
https://reviews.llvm.org/D106593
More information about the llvm-commits
mailing list