[PATCH] D82561: [analyzer][CrossTU] Lower CTUImportThreshold default value
Balázs Kéri via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jun 25 23:59:08 PDT 2020
balazske added a comment.
In D82561#2115578 <https://reviews.llvm.org/D82561#2115578>, @xazax.hun wrote:
> The analyzer inlines small functions within a TU regardless of the thresholds. I think it would be sensible to do the same across TUs in the case we don't do this already.
That means perform a get CTU definition if the TU to be imported (where the function comes from) is small? Otherwise it does not matter how small the function is, it can result in importing of large amount of code. Determining parameters (like "smallness") of the TU is probably not simple.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82561/new/
https://reviews.llvm.org/D82561
More information about the cfe-commits
mailing list