MaskRay wrote: Current behavior: ``` % clang --target=x86_64-linux-musl -fms-volatile -c a.cc clang: warning: argument unused during compilation: '-fms-volatile' [-Wunused-command-line-argument] % clang --target=x86_64-windows -fms-volatile -c a.cc clang: warning: argument unused during compilation: '-fms-volatile' [-Wunused-command-line-argument] ```
Shall we make -f[no-]ms-volatile an error for non-Windows targets? (mark it as `TargetSpecific` and only claim it for `isOSWindows()`). >From >https://learn.microsoft.com/en-us/cpp/build/reference/volatile-volatile-keyword-interpretation?view=msvc-170 > , I assume that /volatile:ms is a legacy behavior that non-Windows OSes >likely don't want to adopt. https://github.com/llvm/llvm-project/pull/74790 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits