rjmccall wrote:

> Okay the problem with using `ASTContext` here is that it creates some 
> recursive includes. ~I can do this by moving the check into `Type.cpp` 
> instead, so this will be function call instead of being inlined.~ This would 
> require a lot of extra stuff so I'm going to defer it unless we _really_ 
> think it needs to be done.

Can you just outline the slow path where you have to actually call the target 
info hook?

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

Reply via email to