<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jun 16, 2016 at 2:44 PM, Davide Italiano <span dir="ltr"><<a href="mailto:dccitaliano@gmail.com" target="_blank">dccitaliano@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Thu, Jun 16, 2016 at 2:42 PM, Xinliang David Li <<a href="mailto:xinliangli@gmail.com">xinliangli@gmail.com</a>> wrote:<br>
> Davide, as Danny noticed, I think the main troublesome patch is r272595.<br>
> There is no need to eliminate that class and statiize all the methods.<br>
> Instead, what is needed is to move legacy pass specific members out of class<br>
> MergedLoadStoreMotion and keep the original class as the 'transformation<br>
> class' that can be shared.<br>
><br>
> Can you revert that change and do the above suggested refactoring -- that<br>
> will only take a few lines, I think.<br>
><br>
<br>
</span>Already reverted, working on a fix now. Sorry for the trouble.<br>
</blockquote></div><br></div><div class="gmail_extra"><br></div><div class="gmail_extra">no problem -- thanks for the quick action :)</div><div class="gmail_extra"><br></div></div>