[PATCH] D121085: [AArch64][GlobalISel] Implement G_SELECT translate to min/max/abs

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 17 15:49:16 PDT 2023


arsenm requested changes to this revision.
arsenm added a comment.
This revision now requires changes to proceed.
Herald added a subscriber: StephenFan.

Is this still relevant?

I think a native min/max matching combiner would be more helpful, the pattern can easily appear during legalization. InstCombine forms min/max intrinsics these days, so producing them at the start of codegen isn't really necessary


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121085



More information about the llvm-commits mailing list