diff options
author | Alex Voicu <alexandru.voicu@amd.com> | 2025-07-28 22:29:31 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2025-07-28 22:29:31 +0100 |
commit | 6bcff9eb13f226b89bb6ebc80bb0f3e80b7868f7 (patch) | |
tree | 3e180eca1c48ee388ff78e1bbf50f01a326c6d75 /mlir/lib/Analysis/DataFlow/ConstantPropagationAnalysis.cpp | |
parent | ced3b90738ff6a4c2f5f264e5085477dc59ffcf8 (diff) | |
download | llvm-6bcff9eb13f226b89bb6ebc80bb0f3e80b7868f7.zip llvm-6bcff9eb13f226b89bb6ebc80bb0f3e80b7868f7.tar.gz llvm-6bcff9eb13f226b89bb6ebc80bb0f3e80b7868f7.tar.bz2 |
[HIPSTDPAR] Add handling for math builtins (#140158)
When compiling in `--hipstdpar` mode, the builtins corresponding to the
standard library might end up in code that is expected to execute on the
accelerator (e.g. by using the `std::` prefixed functions from
`<cmath>`). We do not have uniform handling for this in AMDGPU, and the
errors that obtain are quite arcane. Furthermore, the user-space changes
required to work around this tend to be rather intrusive.
This patch adds an additional `--hipstdpar` specific pass which forwards
to the run time component of HIPSTDPAR the intrinsics / libcalls which
result from the use of the math builtins, and which are not properly
handled. In the long run we will want to stop relying on this and handle
things in the compiler, but it is going to be a rather lengthy journey,
which makes this medium term escape hatch necessary.
The paired change in the run time component is here
<https://github.com/ROCm/rocThrust/pull/551>.
Diffstat (limited to 'mlir/lib/Analysis/DataFlow/ConstantPropagationAnalysis.cpp')
0 files changed, 0 insertions, 0 deletions