https://github.com/Sirraide commented:

I think the general approach of this makes sense. 

Can you add some more tests that involve an expression with this new type as 
the argument to a function call? You’ve taken care of handling that case from 
what I’ve seen, but having tests for that would probably be a good idea.

https://github.com/llvm/llvm-project/pull/89019
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to