[PATCH] D45540: [globalisel][legalizerinfo] Introduce dedicated extending loads and add lowerings for them

Amara Emerson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 18 17:30:56 PDT 2018


aemerson accepted this revision.
aemerson added a comment.
This revision is now accepted and ready to land.

LGTM, with addition of `-verify-machineinstrs` to the tests.



================
Comment at: lib/CodeGen/GlobalISel/LegalizerHelper.cpp:742
       return UnableToLegalize;
-
+    // fallthrough
+  case TargetOpcode::G_SEXTLOAD:
----------------
`LLVM_FALLTHROUGH` here


Repository:
  rL LLVM

https://reviews.llvm.org/D45540





More information about the llvm-commits mailing list