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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
