[PATCH] D102391: [AMDGPU][GlobalISel] Legalize G_ABS

Mirko Brkusanin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 3 07:57:15 PDT 2021


mbrkusanin updated this revision to Diff 349552.
mbrkusanin added a comment.

- Add lowerAbsToMaxSub as a new LegalizerHelper function to be used by AMDGPURegisterBankInfo
- Move default code for lowering G_ABS to lowerAbs to make it more clear there are two ways of legalizing.

Maybe rename lowerAbs to lowerAbsToAshrAddXor or lowerAbsToAddXor?


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

https://reviews.llvm.org/D102391

Files:
  llvm/include/llvm/CodeGen/GlobalISel/LegalizerHelper.h
  llvm/lib/CodeGen/GlobalISel/LegalizerHelper.cpp
  llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp
  llvm/lib/Target/AMDGPU/AMDGPURegisterBankInfo.cpp
  llvm/lib/Target/AMDGPU/SOPInstructions.td
  llvm/test/CodeGen/AMDGPU/GlobalISel/llvm.abs.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D102391.349552.patch
Type: text/x-patch
Size: 12134 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210603/19bbc532/attachment-0001.bin>


More information about the llvm-commits mailing list