[PATCH] D135044: [GlobalISel] Allow prelegalizer combiners to have access to LegalizerInfo.
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Oct 2 18:41:04 PDT 2022
arsenm accepted this revision.
arsenm added a comment.
This revision is now accepted and ready to land.
LGTM with nit
================
Comment at: llvm/lib/Target/AArch64/GISel/AArch64PreLegalizerCombiner.cpp:380-381
MachineIRBuilder &B) const {
- CombinerHelper Helper(Observer, B, KB, MDT);
+ const auto *LI =
+ MI.getParent()->getParent()->getSubtarget().getLegalizerInfo();
+ CombinerHelper Helper(Observer, B, /* IsPreLegalize*/ true, KB, MDT, LI);
----------------
Easier to read this out of the CombinerHelper
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D135044/new/
https://reviews.llvm.org/D135044
More information about the llvm-commits
mailing list