This revision was automatically updated to reflect the committed changes.
Closed by commit rG49c248bd62a3: clang-cl: Remove /Zd flag (authored by thakis).
Herald added a project: clang.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D93458/new/
https:/
rnk accepted this revision.
rnk added a comment.
lgtm
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D93458/new/
https://reviews.llvm.org/D93458
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/list
aeubanks accepted this revision.
aeubanks added a comment.
This revision is now accepted and ready to land.
lgtm, but perhaps Reid should take a look
Comment at: clang/docs/ReleaseNotes.rst:127
+
+- clang-cl's ``/Zd`` flag no longer exist. But ``-gline-tables-only`` still
+ ex
thakis created this revision.
thakis added a reviewer: aeubanks.
Herald added a subscriber: dang.
thakis requested review of this revision.
cl.exe doesn't understand Zd (in either MSVC 2017 or 2019), so neiter
should we. It used to do the same as `-gline-tables-only` which is
exposed as clang-cl f
