[PATCH] D80458: [AArch64][GlobalISel] Enable extending loads combines post-legalization.

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 22 16:06:06 PDT 2020


arsenm added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/GlobalISel/CombinerHelper.h:99-103
+  bool matchCombineExtendingLoads(MachineInstr &MI, PreferredTuple &MatchInfo,
+                                  bool EnableAnyExt = true);
+  bool matchCombineExtendingLoadsNoAnyExt(MachineInstr &MI,
+                                          PreferredTuple &MatchInfo);
   void applyCombineExtendingLoads(MachineInstr &MI, PreferredTuple &MatchInfo);
----------------
This should just fix the combiner to understand checking for legal loads rather than special casing anyext?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80458





More information about the llvm-commits mailing list