<div dir="ltr">Hi,<div><br></div><div>First of all, I agree that the discussion on whether to keep `-passes=...` or `-foo-pass` is a separate discussion and deserves its own RFC thread.</div><div><br></div><div>I don't know of a written rationale for the decision to move forward only with `passes=...`. This was mostly discussed verbally, and no firm decision is yet in place. I believe the high-level motivation was the precision/clarity with which the `passes=...` format can describe a pass pipeline (one example from the recent compatibility patch is that NPM cannot have AA passes arbitrarily interleaved with module/function/loop, it's using an AA pipeline instead).</div><div><br></div><div>I'd suggest having a broader discussion on pros/cons, either now or after the switch to the NPM, as the community prefers.</div><div><br></div><div>Thanks,</div><div>Alina</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jun 25, 2020 at 11:20 AM Chen, Yuanfang <<a href="mailto:Yuanfang.Chen@sony.com">Yuanfang.Chen@sony.com</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">





<div lang="EN-US">
<div class="gmail-m_-1539200509800794168WordSection1">
<p class="MsoNormal">Agree with <a id="gmail-m_-1539200509800794168OWAAM4EE8DDCAFEE74817B5A24B687E9D191D" href="mailto:aeubanks@google.com" target="_blank">
<span style="font-family:Calibri,sans-serif;text-decoration:none">@Arthur Eubanks</span></a> that we could leave the discussion (legacy PM versus new PM) on 1. Option compatibility 2. Pass name compatibility to the future. IMHO, the current issue about test
 correctness parity between legacy and new PM regarding pass name should be done with least disruptive way. Name alias seems to be proper to achieve that.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">- Yuanfang<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal" style="margin-left:0.5in"><b>From:</b> llvm-dev <<a href="mailto:llvm-dev-bounces@lists.llvm.org" target="_blank">llvm-dev-bounces@lists.llvm.org</a>>
<b>On Behalf Of </b>Arthur Eubanks via llvm-dev<br>
<b>Sent:</b> Thursday, June 25, 2020 10:09 AM<br>
<b>To:</b> Roman Lebedev <<a href="mailto:lebedev.ri@gmail.com" target="_blank">lebedev.ri@gmail.com</a>><br>
<b>Cc:</b> llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>>; Alina Sbirlea <<a href="mailto:asbirlea@google.com" target="_blank">asbirlea@google.com</a>><br>
<b>Subject:</b> Re: [llvm-dev] Renaming passes<u></u><u></u></p>
<p class="MsoNormal" style="margin-left:0.5in"><u></u> <u></u></p>
<div>
<div>
<p class="MsoNormal" style="margin-left:0.5in"><u></u> <u></u></p>
</div>
<p class="MsoNormal" style="margin-left:0.5in"><u></u> <u></u></p>
<div>
<div>
<p class="MsoNormal" style="margin-left:0.5in">On Thu, Jun 25, 2020 at 9:59 AM Roman Lebedev <<a href="mailto:lebedev.ri@gmail.com" target="_blank">lebedev.ri@gmail.com</a>> wrote:<u></u><u></u></p>
</div>
<blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in">
<p class="MsoNormal" style="margin-left:0.5in">On Thu, Jun 25, 2020 at 7:48 PM Arthur Eubanks via llvm-dev<br>
<<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br>
><br>
> After talking with some NPM people, I believe the ultimate goal after NPM is enabled by default is to only support `-passes=`, and remove support for `-foo-pass`.<br>
Hm, is there any written rationale behind such a decision?<br>
I would have thought that -passes= is the temporary solution, not the<br>
other way around.<u></u><u></u></p>
</blockquote>
<div>
<p class="MsoNormal" style="margin-left:0.5in">This is really a separate issue that's somewhat orthogonal to the original issue, but someone like asbirlea may be able to chime in more. Maybe a new RFC thread?<u></u><u></u></p>
</div>
<blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in">
<p class="MsoNormal" style="margin-left:0.5in"><br>
> However, until NPM is enabled by default, we still want tests using opt to use the legacy PM by default.<br>
> We could attempt to make `-passes=` work with the legacy PM and have a legacy vs new PM flag, but given the design/syntax of `-passes=` I don't think that's feasible (see llvm/include/llvm/Passes/PassBuilder.h).<br>
> So for making sure everything works with NPM, I think we need to support `-foo-pass` in NPM to be able to run all opt tests against NPM. Then at some point after NPM is enabled by default we can attempt to migrate everything to `-passes=`.<br>
<br>
Roman.<u></u><u></u></p>
</blockquote>
</div>
</div>
</div>
</div>

</blockquote></div>