[clang] [clang-tools-extra] [Clang] Use TargetInfo when deciding is an address space is compatible (PR #115777)

John McCall via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 13 15:13:30 PST 2024


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


More information about the cfe-commits mailing list