[PATCH] D121951: [AMDGPU] Only warn when mixing printf and hostcall
Scott Linder via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 17 14:12:12 PDT 2022
scott.linder added reviewers: sameerds, arsenm, rampitec, b-sumner, kzhuravl, yaxunl.
scott.linder added a comment.
For reference, the error I'm changing to a warning was added as part of https://reviews.llvm.org/D70038, and the issue that made me consider this approach appears when building OCL conformance tests at -O0
Another approach would be to do some more work in Clang or early in the backend to strip out the unused device-libs functions which are calling `__ockl_hostcall_internal`. I started with that thought, but backed off as it seemed the less we could do at -O0 the better. The limitation is also being resolved in later HSA ABI versions, so this would be a legacy path almost as soon as we added it.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D121951/new/
https://reviews.llvm.org/D121951
More information about the llvm-commits
mailing list