[PATCH] D140135: AMDGPU: Try to unfold fneg source when matching legacy fmin/fmax

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 2 18:50:43 PST 2023


arsenm closed this revision.
arsenm added a comment.

36cfe26a5288d99e66c75d82989d154874999b98



================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp:1401
+}
+SDValue AMDGPUTargetLowering::combineFMinMaxLegacyImpl(
+    const SDLoc &DL, EVT VT, SDValue LHS, SDValue RHS, SDValue True,
----------------
foad wrote:
> This function doesn't use False. Might be clearer to take a Swapped flag instead of True and False.
I think it's harder to read if you're considering the caller context


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

https://reviews.llvm.org/D140135



More information about the llvm-commits mailing list