While we might eventually want to disallow allocas that do not have the
alloca-AS set, it seems undesirable to crash on them. Add a cast when
required so that we can support such allocas (at least here).
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
We'll actually going to create the allocas in heap-2-stack in the right address space later but this is an independent problem we need to fix. The original test exposed another one too, should all be better by tomorrow.
Comment Actions
Hi Johannes, can this patch land?
i hit an instance of the issue it fixes, while compiling (attempting to) picongpu using trunk clang for amdgpu.