<div><br></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Apr 23, 2019 at 11:36 PM Petr Hosek via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">phosek added inline comments.<br>
<br>
<br>
================<br>
Comment at: clang/lib/Driver/ToolChains/CommonArgs.h:122<br>
+<br>
+/// \p Flag must be a flag accepted by the driver with its leading '-' removed,<br>
+//     otherwise '-print-multi-lib' will not emit them correctly.<br>
----------------<br>
jroelofs wrote:<br>
> Can we enforce this precondition with an assert? The '-'-prefix-not-there part is easy, but what about the "it's a driver flag" part?<br>
We could, but we would need to pass in the reference to `Driver` so we can use `getOpts().ParseOneArg` to parse the flag. However, we're not doing that in `Multilib` either at the moment even though [the comment says it has to be a valid flag](<a href="https://github.com/llvm/llvm-project/blob/master/clang/include/clang/Driver/Multilib.h#L80" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/blob/master/clang/include/clang/Driver/Multilib.h#L80</a>). I'd prefer to keep this as a pure move and then add the additional checking both here and to `Multilib` as a separate change, would that be fine with you?</blockquote><div dir="auto"><br></div><div dir="auto">Sure, that works.</div><div dir="auto"><br></div><div dir="auto">Jon</div><div dir="auto"><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
<br>
Repository:<br>
  rC Clang<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D61040/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D61040/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D61040" rel="noreferrer" target="_blank">https://reviews.llvm.org/D61040</a><br>
<br>
<br>
<br>
</blockquote></div></div>