https://github.com/efriedma-quic commented:

This looks basically right.  But I'm not sure all the conditions for calling it 
are correct. Is getDeclareMapperVarRef relevant?  Does it matter what scope 
we're inserting the variable into?  (Can the variable be inserted in a 
namespace?  Can it be a local variable?)

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

Reply via email to