[clang] [clang][Sema] Don't warn for implicit uses of builtins in system headers (PR #138205)

Nick Sarnie via cfe-commits cfe-commits at lists.llvm.org
Fri May 2 12:32:05 PDT 2025


sarnex wrote:

I think it's because we are explciitly marking these builtins as requiring a header. If they aren't declared that way then you won't see this warning and won't need this change.

I made the builtins require a header based on feedback from @rnk in the earlier PR, and his suggestion on [my regression fix PR](https://github.com/llvm/llvm-project/pull/136742) to still require the header but follow what we did for `_m_prefetchw`, but it seems the same thing was done there but that was also reverted, so I think they hit the same problem. See [here](https://github.com/llvm/llvm-project/pull/115099) and revert [here](https://github.com/llvm/llvm-project/commit/83ff9d4a34b1e579dd809759d13b70b8837f0cde).

So if I understand the state of affairs correctly if we wan't to prevent the `winnt.h` warning we need to either
1) Not require the header
2) Somehow prevent the warning

Let me know if anyone has suggestions.

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


More information about the cfe-commits mailing list