<div dir="ltr">Sounds good to me. But there's a detail to be dealt with.<div><br></div><div>For out of the box _testing_, having the VS style output be the default will force us to, either:</div><div> - use a script to run the commands (doable, but ugly (except if you need the script anyway, for remote testing. I'm ok with this))</div><div> - or change all the tests to expect one of the output styles (I don't really like this :-) )</div><div><br></div><div>I would like to avoid both of those solutions.</div><div><br></div><div>What do you think about doing something like this:</div><div><br></div><div>Have options get their values in this order:</div><div> - Compiled-in defaults</div><div> - $SANITIZER_OPTIONS (new)</div><div><div> - __tool_default_options() (asan, ubsan, etc)</div></div><div> - $TOOL_OPTIONS (ASAN, UBSAN, etc)</div><div><br></div><div>The rationale is:</div><div> - Compiled-in defaults should be sensible defaults for the platform.</div><div> - $SANITIZER_OPTIONS could set common flags (only, no tool-specific options) for the user/machine</div><div> - __tool_default_options() can be used in a program to override some flags because the test/program expects this</div><div> - $TOOL_OPTIONS is a final chance for the current user, running the program, to override.</div><div><br></div><div>I'm ok with not doing this, and simply waiting until it's more necessary.</div><div>For the PS4, I can just work around it and "hack" our remote run script to always add symbolize_vs_style=false to *_OPTIONS env vars when testing. But it wouldn't be nice for local Windows testing (non-remote), since we would default to vs-style (if _MSC_VER was defined) in some cases.<br></div><div><br></div><div>Any ideas?</div><div><br></div><div>Thanks,</div><div><br></div><div> Filipe</div><div><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jun 4, 2015 at 2:57 PM, Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">REPOSITORY<br>
  rL LLVM<br>
<span class=""><br>
================<br>
Comment at: lib/sanitizer_common/sanitizer_flags.inc:149<br>
@@ -148,1 +148,3 @@<br>
             "Print inlined frames in stacktraces. Defaults to true.")<br>
+COMMON_FLAG(bool, symbolize_vs_style, false,<br>
+            "Print file locations in Visual Studio style (e.g: "<br>
----------------<br>
</span><span class="">samsonov wrote:<br>
> filcab wrote:<br>
> > This should probably default to SANITIZER_WINDOWS instead of false.<br>
> > I think it would be the option that better matches what a user of that platform would expect, no?<br>
> I'd let Reid comment on this. I don't have strong preference about Windows defaults.<br>
</span>Let's default this to `defined(_MSC_VER)`. Thankfully it is very easy to override the default sanitizer options with __asan_default_options(), so I'd rather optimize for first time users of visual studio rather than people like us with big cross-platform projects.<br>
<div class=""><div class="h5"><br>
<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D10113&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=pEOoZ8kdgkqmc4WBx9rTqnlmwqOvNuWHWxknYRokDWw&s=_dMtsIFy8PSdbkNotoQ0eBrHCF_WmTXnS6efjU7s4Sk&e=" target="_blank">http://reviews.llvm.org/D10113</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_settings_panel_emailpreferences_&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=pEOoZ8kdgkqmc4WBx9rTqnlmwqOvNuWHWxknYRokDWw&s=Q6IMJ_MkbdR08Kit43MSeRRtU3cz4Q4It6DrEOSL-Jw&e=" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</div></div></blockquote></div><br></div></div></div>