[llvm] [AMDGPU] Avoid put implicit_def into bundle that break reg's liveness (PR #142563)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 4 20:58:04 PDT 2025


arsenm wrote:

> We cannot include implicit_def into the bundle because inside the bundle implicit def will not count for defined register and any use of this register in the latter program will cause the verification error.

You can and should include implicit_def in the bundle. It does not matter that it's an implicit_def, the liveness of that def should work the same as any other real instruction inside the bundle. 

> if the register it is defining doesn't overlap with other instruction, move the implicit_def into the front of the bundle

You don't really need to move anything. I guess you could handle it this way, but I think it would be more difficult than just treating the implicit_def like any other load def in the bundle 



https://github.com/llvm/llvm-project/pull/142563


More information about the llvm-commits mailing list