<div dir="ltr">Thanks Justin, and thanks to the others for all the useful comments. Landing this, and will switch to post-commit review for the subsequent pass porting as long as its boring and mechanically following this pattern. Naturally, I'll surface anything interesting and get feedback first. Shout if folks see issues or if they start disliking the patterns here.</div><br><div class="gmail_quote"><div dir="ltr">On Fri, Sep 11, 2015 at 10:24 PM Justin Bogner via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Chandler Carruth <<a href="mailto:chandlerc@gmail.com" target="_blank">chandlerc@gmail.com</a>> writes:<br>
> Thanks for the feeback!<br>
><br>
> On Fri, Sep 11, 2015 at 11:47 AM Justin Bogner via llvm-commits <<br>
> <a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br>
><br>
>     A couple of minor comments below, then as long as nobody's opposed to<br>
>     this path it LGTM.<br>
><br>
> Comments addressed, but there is one outstanding question I'd like a better<br>
> answer to before I move forward: how to name the legacy pass manager passes.<br>
><br>
> I don't actually have strong feelings here. Here is the current reasoning I<br>
> see, which doesn't leave a clear choice IMO:<br>
><br>
> - FooWrapperPass is a legacy PM wrapper around the Foo analysis results<br>
> - FooLegacyPass is a legacy PM pass for the Foo transformation<br>
><br>
> Meh. Thoughts?<br>
<br>
Yeah, I don't have strong feelings either. As long as there's some<br>
distinction between the Legacy vs Wrapper naming and it isn't just "toss<br>
a coin and choose a name" I'm fine with using both. It'd also be fine to<br>
standardize on "Legacy", IMO.<br>
<br>
This is good to go in. It's pretty much the best of the solutions that<br>
have been brought up so far, nobody's really objected, and if someone<br>
comes up with something better we can always change it.<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>