[all-commits] [llvm/llvm-project] 9ec668: AMDGPU: Add option to disable CGP division expansion
Matt Arsenault via All-commits
all-commits at lists.llvm.org
Fri Feb 14 11:37:22 PST 2020
Branch: refs/heads/master
Home: https://github.com/llvm/llvm-project
Commit: 9ec668606b3a8d71ca327d9857844fb9c607a54b
https://github.com/llvm/llvm-project/commit/9ec668606b3a8d71ca327d9857844fb9c607a54b
Author: Matt Arsenault <Matthew.Arsenault at amd.com>
Date: 2020-02-14 (Fri, 14 Feb 2020)
Changed paths:
M llvm/lib/Target/AMDGPU/AMDGPUCodeGenPrepare.cpp
M llvm/lib/Target/AMDGPU/AMDGPUGISel.td
M llvm/lib/Target/AMDGPU/AMDGPURegisterBankInfo.cpp
M llvm/lib/Target/AMDGPU/SIInstructions.td
M llvm/test/CodeGen/AMDGPU/GlobalISel/legalize-sdiv.mir
M llvm/test/CodeGen/AMDGPU/GlobalISel/legalize-srem.mir
M llvm/test/CodeGen/AMDGPU/GlobalISel/legalize-udiv.mir
M llvm/test/CodeGen/AMDGPU/GlobalISel/legalize-urem.mir
Log Message:
-----------
AMDGPU: Add option to disable CGP division expansion
The division expansions in AMDGPUCodeGenPrepare can't be relied on for
correctness, since they punt to later optimization and possibly
legalization in some cases. We still need a way to be able to write
tests for the legalizer versions of the expansion. This is mostly for
GlobalISel, since the expected optimzations is expecting aren't
implemented.
The interaction with the flag to expand 64-bit division in the IR is
pretty confusing, but these flags have different purposes.
More information about the All-commits
mailing list