<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Sep 4, 2015 at 1:29 AM, Davide Italiano <span dir="ltr"><<a href="mailto:davide@freebsd.org" target="_blank">davide@freebsd.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Thu, Sep 3, 2015 at 9:20 AM, Rui Ueyama via llvm-commits<br>
<<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br>
> Author: ruiu<br>
> Date: Thu Sep  3 11:20:47 2015<br>
> New Revision: 246780<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=246780&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=246780&view=rev</a><br>
> Log:<br>
> COFF: Ignore /GUARDSYM option.<br>
><br>
> The option is added in MSVC 2015, and there's no documentation about<br>
> what the option is. This patch is to ignore the option for now, so that<br>
> at least LLD is usable with MSVC 2015.<br>
><br>
> Modified:<br>
>     lld/trunk/COFF/Driver.cpp<br>
>     lld/trunk/COFF/Options.td<br>
><br>
> Modified: lld/trunk/COFF/Driver.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/lld/trunk/COFF/Driver.cpp?rev=246780&r1=246779&r2=246780&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/lld/trunk/COFF/Driver.cpp?rev=246780&r1=246779&r2=246780&view=diff</a><br>
> ==============================================================================<br>
> --- lld/trunk/COFF/Driver.cpp (original)<br>
> +++ lld/trunk/COFF/Driver.cpp Thu Sep  3 11:20:47 2015<br>
> @@ -120,6 +120,7 @@ void LinkerDriver::parseDirectives(Strin<br>
>        Config->NoDefaultLibs.insert(doFindLib(Arg->getValue()));<br>
>        break;<br>
>      case OPT_editandcontinue:<br>
> +    case OPT_guardsym:<br>
>      case OPT_throwingnew:<br>
>        break;<br>
>      default:<br>
><br>
<br>
</div></div>Can we emit some kind of diagnostic to let the user know we're ignoring these?<br></blockquote><div> </div><div>We could, but it would be annoying because the option is present in .drectve sections of object files, and users are not aware of the presence of the option.</div></div></div></div>