[PATCH] D52798: [cl-compat] Change /JMC from unsupported to ignored.

2018-10-03 Thread Hans Wennborg via Phabricator via cfe-commits
hans added a comment. Thanks for fixing! Please also update the test in test/Driver/cl-options.c and include me on clang-cl code reviews. Repository: rL LLVM https://reviews.llvm.org/D52798 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D52798: [cl-compat] Change /JMC from unsupported to ignored.

2018-10-02 Thread Zachary Turner via Phabricator via cfe-commits
This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit rL343629: [cl-compat] Change /JMC from unsupported to ignored. (authored by zturner, committed by ). Herald added a subscrib

[PATCH] D52798: [cl-compat] Change /JMC from unsupported to ignored.

2018-10-02 Thread Reid Kleckner via Phabricator via cfe-commits
rnk added a comment. lgtm Can you file a bug about implementing /JMC and link it from the commit message? https://reviews.llvm.org/D52798 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-com

[PATCH] D52798: [cl-compat] Change /JMC from unsupported to ignored.

2018-10-02 Thread Zachary Turner via Phabricator via cfe-commits
zturner created this revision. zturner added reviewers: rnk, thakis. Herald added subscribers: JDevlieghere, aprantl. This command line option doesn't really affect generated code, only generated debug info, and it's set by default in newer versions of VS, so it's annoying to have to turn it off