[PATCH] D95876: [clang-cl] Remove the /fallback option

2021-02-04 Thread Hans Wennborg via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG6625680a581c: [clang-cl] Remove the /fallback option (authored by hans). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95876/new/ https://reviews.llvm.org/D

[PATCH] D95876: [clang-cl] Remove the /fallback option

2021-02-03 Thread Hans Wennborg via Phabricator via cfe-commits
hans added a comment. Please take another look. Comment at: clang/lib/Driver/ToolChains/MSVC.cpp:669 const InputInfoList &Inputs, const ArgList &Args, const char *LinkingOutput) const { ArgStringList CmdArgs; thakis wrote: > Doesn't this whole func

[PATCH] D95876: [clang-cl] Remove the /fallback option

2021-02-03 Thread Hans Wennborg via Phabricator via cfe-commits
hans updated this revision to Diff 321080. hans marked an inline comment as done. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95876/new/ https://reviews.llvm.org/D95876 Files: clang/docs/ReleaseNotes.rst clang/docs/UsersManual.rst clang/include/clang/Basic/DiagnosticDriverKinds.t

[PATCH] D95876: [clang-cl] Remove the /fallback option

2021-02-02 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. Probably get a short release notes entry too. Comment at: clang/lib/Driver/ToolChains/MSVC.cpp:669 const InputInfoList &Inputs, const ArgList &Args, const char *LinkingOutput) const { ArgStringList CmdArgs; Doesn't this whol

[PATCH] D95876: [clang-cl] Remove the /fallback option

2021-02-02 Thread Hans Wennborg via Phabricator via cfe-commits
hans created this revision. hans added reviewers: rnk, thakis. Herald added a subscriber: dang. Herald added a reviewer: jansvoboda11. hans requested review of this revision. Herald added a project: clang. As discussed in https://lists.llvm.org/pipermail/cfe-dev/2021-January/067524.html It doesn