[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