[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