[PATCH] D26781: [LSR] Canonicalize formula and put recursive Reg related with current loop in ScaledReg.

Wed May 17 11:47:38 PDT 2017

I am guessing we are missing a call to canonicalize after we changed a formula.

> On May 17, 2017, at 11:45 AM, Wei Mi <wmi at google.com> wrote:
> 
> Hi Anna,
> 
> Sorry to cause the regression. I am looking at the testcase right now
> you provided in the PR. I will revert the change if I cannot find a
> quick fix.
> 
> Wei.
> 
> On Wed, May 17, 2017 at 10:54 AM, Anna Thomas via Phabricator
> <reviews at reviews.llvm.org> wrote:
>> anna added a comment.
>> 
>> This patch caused an assertion failure as filed in https://bugs.llvm.org/show_bug.cgi?id=33077
>> @wmi: Could you please revert this change?
>> 
>> 
>> Repository:
>>  rL LLVM
>> 
>> https://reviews.llvm.org/D26781
>> 
>> 
>> 