[AMDGPU] Remove function with incompatible features
authorpvanhout <pierre.vanhoutryve@amd.com>
Fri, 10 Feb 2023 08:32:51 +0000 (09:32 +0100)
committerpvanhout <pierre.vanhoutryve@amd.com>
Tue, 21 Feb 2023 09:42:39 +0000 (10:42 +0100)
commit8e68c1204580d0bb57001929d3345af80b27ac01
treedeb21ce2a824ae4bc8fa373fb5cc530e4706e395
parent33c29a82a9b3b8a0354a5b5bd75b462505602107
[AMDGPU] Remove function with incompatible features

Adds a new pass that removes functions
if they use features that are not supported on the current GPU.

This change is aimed at preventing crashes when building code at O0 that
uses idioms such as `if (ISA_VERSION >= N) intrinsic_a(); else intrinsic_b();`
where ISA_VERSION is not constexpr, and intrinsic_a is not selectable
on older targets.
This is a pattern that's used all over the ROCm device libs. The main
motive behind this change is to allow code using ROCm device libs
to be built at O0.

Note: the feature checking logic is done ad-hoc in the pass. There is no other
pass that needs (or will need in the foreseeable future) to do similar
feature-checking logic so I did not see a need to generalize the feature
checking logic yet. It can (and should probably) be generalized later and
moved to a TargetInfo-like class or helper file.

Reviewed By: arsenm, Joe_Nash

Differential Revision: https://reviews.llvm.org/D139000
llvm/include/llvm/MC/MCSubtargetInfo.h
llvm/lib/Target/AMDGPU/AMDGPU.h
llvm/lib/Target/AMDGPU/AMDGPURemoveIncompatibleFunctions.cpp [new file with mode: 0644]
llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
llvm/lib/Target/AMDGPU/CMakeLists.txt
llvm/test/CodeGen/AMDGPU/GlobalISel/dummy-target.ll
llvm/test/CodeGen/AMDGPU/llc-pipeline.ll
llvm/test/CodeGen/AMDGPU/remove-incompatible-functions.ll [new file with mode: 0644]