diff options
| author | Martin Storsjö <martin@martin.st> | 2023-10-02 13:22:23 +0300 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-10-02 13:22:23 +0300 |
| commit | 7bc09a471fbc274d8632d1379d4134bec63fecc4 (patch) | |
| tree | 1fe8586ca3b7dc52ab28f8e3e609e0310c06bf22 /flang/lib/Frontend/CompilerInvocation.cpp | |
| parent | b1295dd5c923c6828775406b4063de1531fd4782 (diff) | |
| download | llvm-7bc09a471fbc274d8632d1379d4134bec63fecc4.zip llvm-7bc09a471fbc274d8632d1379d4134bec63fecc4.tar.gz llvm-7bc09a471fbc274d8632d1379d4134bec63fecc4.tar.bz2 | |
[compiler-rt] Reinstate removal of CRT choice flags from CMAKE_*_FLAGS* (#67935)
This reverts one part of commit
9f4dfcb795bb0ecf9944553f49371164801cd83f, with a modified comment added
about the code.
Ideally, this would only be reinstated temporarily - but given the
situation in vcpkg, it looks likely that they would keep passing the
duplicate options for quite some time. The conflicting CRT choice
usually are benign but only would cause warnings about one option
overriding the other, if passing e.g. "/MDd /MT".
However when vcpkg currently sets these options in CMAKE_*_FLAGS_DEBUG,
it passes the redundant option /D_DEBUG; thus the compiler finally ends
up with e.g. "/D_DEBUG /MDd /MT", which has the effect of defining
_DEBUG while using a release mode CRT, which allegedly breaks the build.
There's a PR up for removing this redundant /D_DEBUG option in vcpkg in
https://github.com/microsoft/vcpkg/pull/34123. With that in place, this
change wouldn't be strictly needed.
Diffstat (limited to 'flang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions