<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Mar 10, 2022 at 12:25 AM Fangrui Song via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">MaskRay added a comment.<br>
<br>
In D120455#3355507 <<a href="https://reviews.llvm.org/D120455#3355507" rel="noreferrer" target="_blank">https://reviews.llvm.org/D120455#3355507</a>>, @ab wrote:<br>
<br>
> In D120455#3352886 <<a href="https://reviews.llvm.org/D120455#3352886" rel="noreferrer" target="_blank">https://reviews.llvm.org/D120455#3352886</a>>, @dexonsmith wrote:<br>
><br>
>> I don't have a strong opinion; IMO `-mllvm` options shouldn't get used (except by compiler developers) since they're unstable, although I'm sure they get used a little anyway. IMO, if this makes it easier for the intended users (compiler developers) then there isn't a real downside to it.<br>
>><br>
>> @ab @t.p.northover @Gerolf , any thoughts from you?<br>
><br>
> Yeah, this seems reasonable, but no strong opinions either.  I totally empathize with the concerns, and ideally we wouldn't have all this exposed surface, but I'm not sure this meaningfully changes that.<br>
<br>
Thanks for the feedback so far!<br>
<br>
@echristo Do you still have concerns with this specific change after my updated summary?<br></blockquote><div><br></div><div>I mean, yes, but I'm also happy to be outvoted here :)</div><div><br></div><div>Thanks for all of the discussion, it's really appreciated!</div><div><br></div><div>-eric </div></div></div>