dzhidzhoev wrote:

> I think it sounds roughly plausible - the perf issue I wouldn't /expect/ to 
> be a problem, there's probably a lot more types (in a full debug info build 
> at least) than functions (hmm, well, actually hard to say - guess it could go 
> either way) & I guess it's well enough optimized it hopefully won't show up 
> in profiles or the like.

Let's try simpler solution https://github.com/llvm/llvm-project/pull/165032. If 
it works out, we won't need to support different llvm::Functions referring to 
the same DISubprogram in DwarfDebug. Uniquing DISubprograms may be a nice idea 
still.

https://github.com/llvm/llvm-project/pull/142166
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to