[PATCH] D23237: [MSSA] Fix PR28880 by fixing stack popping behavior.

Sebastian Pop via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 8 07:58:01 PDT 2016


Thanks Daniel for fixing the MSSA bug.
I have committed https://reviews.llvm.org/rL278010 to enable gvn-hoist
by default.

On Mon, Aug 8, 2016 at 1:40 AM, Daniel Berlin <dberlin at dberlin.org> wrote:
> Hmmm. Some of the bots are failing, but none are failing any actual tests ,
> and the stdio logs are completely unhelpful except to say it waited 1200
> seconds for something to exit.
>
> I have run ninja check on the same set of platforms, and can't reproduce the
> same issue, so hopefully this is just flakiness from something else.
>
> If not, i'll revert.
>
> On Sun, Aug 7, 2016 at 9:52 PM, Phabricator <reviews at reviews.llvm.org>
> wrote:
>>
>> This revision was automatically updated to reflect the committed changes.
>> Closed by commit rL277978: [MSSA] Fix PR28880 by fixing use optimizer's
>> lower bound tracking behavior. (authored by dannyb).
>>
>> Changed prior to commit:
>>   https://reviews.llvm.org/D23237?vs=67088&id=67118#toc
>>
>> Repository:
>>   rL LLVM
>>
>> https://reviews.llvm.org/D23237
>>
>> Files:
>>   llvm/trunk/lib/Transforms/Utils/MemorySSA.cpp
>>   llvm/trunk/test/Transforms/Util/MemorySSA/pr28880.ll
>>
>


More information about the llvm-commits mailing list