[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