[clang] [clang][DebugInfo] Add inlined subprogram metadata for compiler built-ins (PR #189969)

[Orlando Cazalet-Hyams via cfe-commits]

OCHyams wrote:

The DW_AT_artificial usage makes sense.

I can sort of see the case that it may be useful to show in profilers/other tools from a completeness angle, though I'm not sure how useful it is in practice, but I may be underestimating the impact.

Could there any concern over the maintenance of how to categorise new/changed builtins? Could this lead to inconsistent behaviour, if it's easy to forget to update this in the future - is there a way to ensure that's caught by tests?

> This debug info treatment is similar to how Clang already represents implicitly declared functions (e.g. C++ implicit destructors), as well as trap and sanitiser function calls.

Ah right, that helps that there's prior art.



https://github.com/llvm/llvm-project/pull/189969