<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Mar 21, 2015 at 1:30 PM, Paul Robinson <span dir="ltr"><<a href="mailto:Paul_Robinson@playstation.sony.com" target="_blank">Paul_Robinson@playstation.sony.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">In <a href="http://reviews.llvm.org/D8484#144547" target="_blank">http://reviews.llvm.org/D8484#144547</a>, @emaste wrote:<br>
<br>
> In <a href="http://reviews.llvm.org/D8484#144353" target="_blank">http://reviews.llvm.org/D8484#144353</a>, @probinson wrote:<br>
><br>
> > Please disable this for PS4?  Our toolchain isn't ready to support this.<br>
><br>
><br>
> Presumably you wouldn't pass -gsplit-dwarf to clang when building for PS4 though?<br>
<br>
<br>
</span>*I* wouldn't, but licensees love to try stuff out even if we don't claim it<br>
works.  Currently, adding -gsplit-dwarf to a command line silently has<br>
no effect.  With your patch, I get:<br>
<br>
clang.exe: error: unable to execute command: program not executable<br>
<br>
which will inevitably cause a support call and cost us time and effort.<br>
I'd prefer to preserve the "silently has no effect" behavior, at least until<br></blockquote><div><br>Seems that'd be likely to cause a support call too - but in any case, I'll advocate for some error, any error, rather than silently ignoring the flag.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
we are able to support this properly.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
<a href="http://reviews.llvm.org/D8484" target="_blank">http://reviews.llvm.org/D8484</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</div></div></blockquote></div><br></div></div>