[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