sameerds abandoned this revision.
sameerds added a comment.
Abandoned in favour of enabling the workarounds in the AMDGPU backend:
https://reviews.llvm.org/D81211
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78900/new/
https://reviews.llvm.org/D7
sameerds marked an inline comment as done.
sameerds added inline comments.
Comment at: clang/lib/Driver/ToolChains/HIP.cpp:158
Args.MakeArgString(Twine("-filetype=") + (OutputIsAsm ? "asm" : "obj")));
+ LlcArgs.push_back("--amdgpu-enable-structurizer-workarounds");
sameerds added inline comments.
Comment at: clang/lib/Driver/ToolChains/HIP.cpp:158
Args.MakeArgString(Twine("-filetype=") + (OutputIsAsm ? "asm" : "obj")));
+ LlcArgs.push_back("--amdgpu-enable-structurizer-workarounds");
arsenm wrote:
> We should just
arsenm added inline comments.
Comment at: clang/lib/Driver/ToolChains/HIP.cpp:158
Args.MakeArgString(Twine("-filetype=") + (OutputIsAsm ? "asm" : "obj")));
+ LlcArgs.push_back("--amdgpu-enable-structurizer-workarounds");
We should just flip this in the
sameerds created this revision.
Herald added subscribers: llvm-commits, cfe-commits, kerbowa, hiraditya, t-tye,
tpr, dstuttard, yaxunl, nhaehnle, wdng, jvesely, kzhuravl, arsenm.
Herald added projects: clang, LLVM.
sameerds added reviewers: yaxunl, scchan, arsenm, rampitec, dstuttard.
The Strucur