[PATCH] D105433: [AArch64][GlobalISel] Lower vector types for min/max

Irina Dobrescu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 7 01:07:23 PDT 2021


Rin added inline comments.


================
Comment at: llvm/lib/Target/AArch64/GISel/AArch64LegalizerInfo.cpp:173
+      .lowerIf([=](const LegalityQuery &Q) { return Q.Types[0].isScalar(); })
+      .lowerIf([=](const LegalityQuery &Q) { return Q.Types[0].isVector(); });
 
----------------
aemerson wrote:
> paquette wrote:
> > I think you can use the helper functions in LegalizerInfo.h here to make this a little cleaner:
> > 
> > ```
> >  .lowerIf(any(isScalar(0), isVector(0)))
> > ```
> At this point, the rule just becomes equivalent to `lower()`.
Good point, I'll just make it lower()


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105433



More information about the llvm-commits mailing list