rnk accepted this revision.
rnk added a reviewer: rnk.
rnk added a comment.
lgtm
Surely this is going to break something, but let's throw the switch and find
out.
http://reviews.llvm.org/D20347
___
cfe-commits mailing list
cfe-commits@lists.llvm.o
thakis added a comment.
probably at least the "the XOR with RSP/EBP/ESP" bit still (and maybe EH
function upgrades instead of bailing)
Comment at: lib/Driver/Tools.cpp:9990
@@ +9989,3 @@
+ /*default=*/false))
+CmdArgs.push_back("/GS-");
+
hans added a comment.
Is this waiting for anything more now that http://reviews.llvm.org/D20346 has
landed?
http://reviews.llvm.org/D20347
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-co
etienneb added a comment.
This patch needs land after http://reviews.llvm.org/D20346.
thx for the review.
http://reviews.llvm.org/D20347
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commi
etienneb updated this revision to Diff 58502.
etienneb marked an inline comment as done.
etienneb added a comment.
more tests
http://reviews.llvm.org/D20347
Files:
include/clang/Driver/CLCompatOptions.td
lib/Driver/Tools.cpp
test/Driver/cl-fallback.c
test/Driver/cl-options.c
Index: tes