[clang] [llvm] [CloneFunction][DebugInfo] Avoid cloning DILocalVariables of inlined functions (PR #75385)

[Duncan P. N. Exon Smith via cfe-commits]

dexonsmith wrote:

I remember discovering that there are effectively two kinds of DISubprogram already... one for declarations (which usually get uniqued/deduped) and another for definitions (which I believe never do... IIRC they are always "distinct").

I imagine it would be possible/good to separate into two classes. If I'd understood there were two uses I would have done so originally. (The main effort is updating testcases.)

Even without two classes, you might be able to clean up the type graph, knowing that there are in fact two ways DISubprogram is used.

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