[clang] [libcxx] [Clang] Add __builtin_common_reference (PR #121199)

Nikolas Klauser via cfe-commits cfe-commits at lists.llvm.org
Tue May 27 07:43:38 PDT 2025


philnik777 wrote:

> This seems reasonable to me? I'm not a huge fan of how much is being put into `SemaTemplate.cpp` to support all the builtins though. I'm wondering if we need to split that up in a follow-up and have a `SemaTemplateBuiltins` for all of these to live.

I can take a look at splitting things up in a follow-up. Shouldn't be too complicated. I do wonder whether it's worth the split, since it's a relatively small amount of code. OTOH it's a very clean split...

> Please let Aaron/others take a poke at this, and make sure there are a few more Libc++ reviewers to make sure you're all in agreement with how this works (I see the convo that is happening).
> 
> Also, should we loop in libstdc++ folks here?

IDK whether they have any thought on this, but certainly doesn't hurt. CC @jwakely


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


More information about the cfe-commits mailing list