[PATCH] D60433: [AArch64][GlobalISel] Add legalization for some vector G_LSR and G_ASHR

Amara Emerson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 9 09:51:32 PDT 2019


aemerson marked an inline comment as done.
aemerson added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64LegalizerInfo.cpp:277
+  getActionDefinitionsBuilder(G_TRUNC).legalIf(
+      [=](const LegalityQuery &Query) { return true; });
+
----------------
paquette wrote:
> Not for this patch, but in general, I wonder if we should just have a `.alwaysLegal()` or something.
Yes, that's a good idea.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D60433





More information about the llvm-commits mailing list