[clang] [Clang] __has_builtin should return false for aux triple builtins (PR #121839)

Nick Sarnie via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 7 07:11:50 PST 2025


sarnex wrote:

> It is not quite trying to fix a problem:). In hipstdpar mode there's no host/device segregation, and it is possible to codegen IR for host only bits (e.g., parts of the stdlib). We delay resolving whether a construct is or isn't viable to the ME, and just place in these special stubs where the otherwise unsupported construct (in this case a builtin) is used. Then, iff it turns out that the code path that reaches the unsupported construct is accessible, we error out - please see https://reviews.llvm.org/D155850 and https://reviews.llvm.org/D155856 for historical context.
> 
> I don't quite think that this change interacts with that at all (I'll look again), but it might interact in some subtle way with offload languages such as e.g. HIP which jump through some hoops to keep the AST consistent, so I've added @yxsamliu to this review.

Ah, thanks for the clarification for for adding reviewers!

https://github.com/llvm/llvm-project/pull/121839


More information about the cfe-commits mailing list