[clang] [clang] Add deprecation warning for `-Ofast` driver option (PR #98736)
Nico Weber via cfe-commits
cfe-commits at lists.llvm.org
Fri Sep 6 05:23:55 PDT 2024
nico wrote:
> Somehow `clang.exe -Ofast -O2 -### -Werror C:\src\llvm-project\clang\test\Driver\Ofast.c 2>&1` is not producing any output for you, despite `-###` being present. I'm not sure why, but all windows buildbots seem to be happy with this patch. Can you investigate further?
Finally coming back to this. The problem is that I run tests without vcvarsall.bat run, i.e. I don't have MSVC's tools on PATH. `env -i bin/clang/exe -Ofast -O2 -### -Werrror clang/test/Driver/Ofast.c` then prints "clang: error: unable to find a Visual Studio installation; try running Clang from a developer prompt [-Werror,-Wmsvc-not-found]". It looks like this is the only test that
a) schedules to run the linker and
b) uses -Werror
Just adding a `-c` on line 6 (so that the driver doesn't try to discover a linker binary) makes the test pass on that bot. I'm guessing adding that is fine? Probably also for the other run lines in the test?
(Alternatively, `-fuse-ld=lld` would likely do the trick too, but from what I can tell the test isn't actually trying to test linker-related things.)
https://github.com/llvm/llvm-project/pull/98736
More information about the cfe-commits
mailing list