MaxEW707 wrote: I didn't realize ARM64EC was supported by clang.
You are missing one `#if defined(__x86_64__) && !defined(__arm64ec__)` check here, https://github.com/llvm/llvm-project/blob/main/clang/lib/Headers/intrin0.h#L47. Doesn't appear to be anyway to comment on lines that aren't in the diff view on github :(. > If we actually need some of these intrinsics in ARM64EC mode, we can revisit > later. For `intrin0.h` looks like we are missing a some intrinsics that msvc stl does use. Adding `__arm64ec__` to the following lines should match `intrin0.h` that is shipped with msvc 1939. https://github.com/llvm/llvm-project/blob/main/clang/lib/Headers/intrin0.h#L30 https://github.com/llvm/llvm-project/blob/main/clang/lib/Headers/intrin0.h#L86 Since we are intending to revisit adding missing intrinsics later I will do that over the weekend when I have some free time if you decide not to do it in this PR :). https://github.com/llvm/llvm-project/pull/87717 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
