[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