[PATCH] D149130: [PartialInlining] Fix incorrect costing when IR has unreachable BBs
Vedant Paranjape via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Apr 25 20:42:30 PDT 2023
vedant-amd added a comment.
In D149130#4296535 <https://reviews.llvm.org/D149130#4296535>, @fhahn wrote:
> If the block is unreachable, how come it is even considered for outlining?
I think there's simply no check in place, https://github.com/llvm/llvm-project/blob/8fc1764ef3d18b65b52f82ca4a6bf56ac024e589/llvm/lib/Transforms/IPO/PartialInlining.cpp#L1182
It just checks if the BB can be inlined, if not it marks it for outlining.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D149130/new/
https://reviews.llvm.org/D149130
More information about the llvm-commits
mailing list