[PATCH] D40701: [ARM][DAG] Reenable post-legalize store merge

Nirav Davé via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 1 13:40:49 PST 2017


... and AArch64 was working correctly before this.

On Fri, Dec 1, 2017 at 4:40 PM, Nirav Davé <niravd at google.com> wrote:

> Let me back out the last commit. Presumably the constant store merge fix I
> made was wrong.
>
> On Fri, Dec 1, 2017 at 4:33 PM, Evgeny Astigeevich via Phabricator <
> reviews at reviews.llvm.org> wrote:
>
>> eastig added a comment.
>>
>> Nirav,
>>
>> There are stability and correctness issues on AArch64. The similar issues
>> might exist on other ARM targets. Could you please disable
>> MergeConsecutiveStores for all ARM targets including AArch64?
>>
>> Thanks,
>> Evgeny
>>
>>
>> Repository:
>>   rL LLVM
>>
>> https://reviews.llvm.org/D40701
>>
>>
>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171201/c6dce3df/attachment.html>


More information about the llvm-commits mailing list