[PATCH] D63043: [ELF][PPC] Simplify {read,write}FromHalf16

Sean Fertile via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 11 06:13:21 PDT 2019


sfertile added a comment.

In D63043#1537919 <https://reviews.llvm.org/D63043#1537919>, @ruiu wrote:

> Let me revert this change.
>
> Sean, feel free to revert a change if it broke buildbots like this. You don't have to go through a pre-commit code review for a revert change that fixes buildbots.


Thanks. I tried reverting the breaking commit and a commit that built on top of it last week and was able to fix the PowerPC buildbot but broke all the X86 bots in the process. Unfortunately I don't have access to an X86 machine that can build llvm/lld right now so I backed out that change and ask @MaskRay to fix the bot.

In D63043#1537942 <https://reviews.llvm.org/D63043#1537942>, @MaskRay wrote:

> In D63043#1537919 <https://reviews.llvm.org/D63043#1537919>, @ruiu wrote:
>
> > Let me revert this change.
> >
> > Sean, feel free to revert a change if it broke buildbots like this. You don't have to go through a pre-commit code review for a revert change that fixes buildbots.
>
>
> This revision is probably not the broken change. I'll check in a minute.


Your right, I believe it was  commits `729111cf1824159bb4dd331cab8a829eab30313f` and `f49f58527a6d8147524d8d6f2eb1feb70f856292` that need to be pulled out.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D63043





More information about the llvm-commits mailing list