<div dir="ltr">Sounds good.  Clang and LLVM have some support for -fstack-protector.  I wonder how much work it would be to make it compatible with /GS.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Sep 17, 2013 at 4:27 PM, Hans Wennborg <span dir="ltr"><<a href="mailto:hans@hanshq.net" target="_blank">hans@hanshq.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: hans<br>
Date: Tue Sep 17 18:27:39 2013<br>
New Revision: 190900<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=190900&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=190900&view=rev</a><br>
Log:<br>
clang-cl: ignore ths /GS- flag<br>
<br>
The /GS- flag is used to turn off run-time buffer security checks (/GS).<br>
Since no such checks are enabled in the first place, I think we should just<br>
ignore this flag.<br>
<br>
Modified:<br>
    cfe/trunk/include/clang/Driver/CLCompatOptions.td<br>
    cfe/trunk/test/Driver/cl-options.c<br>
<br>
Modified: cfe/trunk/include/clang/Driver/CLCompatOptions.td<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Driver/CLCompatOptions.td?rev=190900&r1=190899&r2=190900&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Driver/CLCompatOptions.td?rev=190900&r1=190899&r2=190900&view=diff</a><br>

==============================================================================<br>
--- cfe/trunk/include/clang/Driver/CLCompatOptions.td (original)<br>
+++ cfe/trunk/include/clang/Driver/CLCompatOptions.td Tue Sep 17 18:27:39 2013<br>
@@ -140,6 +140,7 @@ def _SLASH_TP : CLCompileFlag<"TP">, Hel<br>
<br>
 def _SLASH_analyze_ : CLIgnoredFlag<"analyze-">;<br>
 def _SLASH_errorReport : CLIgnoredJoined<"errorReport">;<br>
+def _SLASH_GS_ : CLIgnoredFlag<"GS-">;<br>
 def _SLASH_nologo : CLIgnoredFlag<"nologo">;<br>
 def _SLASH_Ob1 : CLIgnoredFlag<"Ob1">;<br>
 def _SLASH_Ob2 : CLIgnoredFlag<"Ob2">;<br>
<br>
Modified: cfe/trunk/test/Driver/cl-options.c<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/cl-options.c?rev=190900&r1=190899&r2=190900&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/cl-options.c?rev=190900&r1=190899&r2=190900&view=diff</a><br>

==============================================================================<br>
--- cfe/trunk/test/Driver/cl-options.c (original)<br>
+++ cfe/trunk/test/Driver/cl-options.c Tue Sep 17 18:27:39 2013<br>
@@ -97,7 +97,7 @@<br>
 // Ignored options. Check that we don't get "unused during compilation" errors.<br>
 // (/Zs is for syntax-only, /WX is for -Werror)<br>
 // RUN: %clang_cl /Zs /WX /analyze- /errorReport:foo /nologo /Ob1 /Ob2 -- %s<br>
-// RUN: %clang_cl /Zs /WX /Zc:forScope /Zc:wchar_t /w12345 /wd1234 /RTC1 -- %s<br>
+// RUN: %clang_cl /Zs /WX /Zc:forScope /Zc:wchar_t /w12345 /wd1234 /RTC1 /GS- -- %s<br>
<br>
 // Ignored options and compile-only options are ignored for link jobs.<br>
 // RUN: touch %t.obj<br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</blockquote></div><br></div>