[PATCH] D150667: [GlobalIsel][X86] Legalize G_BSWAP

Bjorn Pettersson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 16 07:44:23 PDT 2023


bjope added a comment.

In D150667#4346179 <https://reviews.llvm.org/D150667#4346179>, @bjope wrote:

> This seems to have broken lots of tests.
> I reverted your patch here: https://github.com/llvm/llvm-project/commit/e016fe817c8e4774c3996314b9519f8d856cc616
>
> See for example this build bot:
>
>   https://lab.llvm.org/buildbot/#/builders/16/builds/48152

Solution is probably to use

  getActionDefinitionsBuilder(G_BSWAP)

instead of having the initializer list with one element.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150667



More information about the llvm-commits mailing list