[PATCH] D81710: [amdgpu] Skip OR combining on 64-bit integer before legalizing ops.

Michael Liao via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 12 12:02:45 PDT 2020


hliao added a comment.

In D81710#2090357 <https://reviews.llvm.org/D81710#2090357>, @rampitec wrote:

> In D81710#2090255 <https://reviews.llvm.org/D81710#2090255>, @arsenm wrote:
>
> > In D81710#2090252 <https://reviews.llvm.org/D81710#2090252>, @rampitec wrote:
> >
> > > What's the rational and practical test?
> >
> >
> > The full or is a better canonical form and splitting it will interfere with other combines
>
>
> OK, but is there any test?


There's one existing test benefitting from that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81710





More information about the llvm-commits mailing list