OCHyams wrote:

> Maybe you meant to request reviews instead of setting assignments…? 🤔

I did indeed, oops

> This makes sense to me; please add a comment as to why there are two paths 
> for DISubprogram production in the code added though.

Will do

> Would it make sense to push this logic into EmitFunctionDecl, or is that too 
> complicated?

I can have a look

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

Reply via email to