[PATCH] D140046: [PowerPC] Fix up memory ordering after combining BV to a load

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

In D140046#4002234 <https://reviews.llvm.org/D140046#4002234>, @nemanjai wrote:

> In D140046#4002193 <https://reviews.llvm.org/D140046#4002193>, @lebedev.ri wrote:
>
>> In D140046#4002169 <https://reviews.llvm.org/D140046#4002169>, @lebedev.ri wrote:
>>
>>> @nemanjai looks like i may have some rather unfortunate news:
>>> https://lab.llvm.org/buildbot/#/builders/93/builds/12560
>>> I'm waiting for https://lab.llvm.org/buildbot/#/builders/93/builds/12561 to finish, and if it suddenly doesn't succeed, revert the recommit...
>>
>> Silver-lining: little-endian PPC bots appear to be green: https://lab.llvm.org/buildbot/#/builders/121 https://lab.llvm.org/buildbot/#/builders/36
>> So the remaining problem is limited to big-endian.
>
> @lebedev.ri Did your original commit have this problem on the big endian bot?

Yup: https://lab.llvm.org/buildbot/#/builders/93/builds/12251
To be honest, i didn't see that particular report :/


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D140046/new/

https://reviews.llvm.org/D140046