<div dir="ltr">Thanks for the feeback!<br><br><div class="gmail_quote"><div dir="ltr">On Fri, Sep 11, 2015 at 11:47 AM 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"><br>
A couple of minor comments below, then as long as nobody's opposed to<br>
this path it LGTM.<br></blockquote><div><br></div><div>Comments addressed, but there is one outstanding question I'd like a better answer to before I move forward: how to name the legacy pass manager passes.</div><div><br></div><div>I don't actually have strong feelings here. Here is the current reasoning I see, which doesn't leave a clear choice IMO:</div><div><br></div><div>- FooWrapperPass is a legacy PM wrapper around the Foo analysis results</div><div>- FooLegacyPass is a legacy PM pass for the Foo transformation</div><div><br></div><div>Meh. Thoughts?</div></div></div>