[PATCH] D65360: [AArch64][GlobalISel] Eliminate redundant G_ZEXT when the source is implicitly zext-loaded
Jessica Paquette via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 29 09:16:03 PDT 2019
paquette accepted this revision.
paquette added a comment.
This revision is now accepted and ready to land.
I think that for -O0 this makes sense.
(As for another combiner round, I wonder if it would make sense to have a "minimal combiner" for -O0 which only handles //extremely// simple transformations? Or is just adding a pass too much compile time overhead, even if it results in fewer instructions and thus less work for later passes?)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65360/new/
https://reviews.llvm.org/D65360
More information about the llvm-commits
mailing list