[llvm] [AMDGPU] Use AllowLDSDMA=false in non-hazard passes (part 2) (PR #222086)
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Sun Sep 13 09:28:00 PDT 2026
arsenm wrote:
> > I'm confused. isVALU should unequivocally return true for DMA operations. It is a question of encoding, and all of these cases are VALU. The cases that care about wait state are aberrant and should do something else than checking isVALU
>
> I misunderstood your initial comment. Is it better to drop the this AllowLDSDMA=false sweep and instead fix those aberrant call sites with isLDSDMA/a dedicated helper?
Yes. isVALU should not have an argument, and be true for LDS DMA instructions. The problems with wait insertion are not transferable to most anywhere else
https://github.com/llvm/llvm-project/pull/222086
More information about the llvm-commits
mailing list