[PATCH] D20728: AMDGPU: Disable AMDGPUPromoteAlloca pass for shader calling conventions.

Nicolai Hähnle via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 1 06:46:37 PDT 2016


nhaehnle added a comment.

The problem is that Mesa currently lies about which data lives in LDS, in particular for tessellation shader inputs and outputs. A proper fix would first need to take care of that.

In general it's unclear to me how Mesa and LLVM should communicate about how LDS is used. Can we get a guarantee e.g. that LLVM-reserved LDS memory will always be placed after any LDS object in the initial IR?

I vote to fix the current issue for now (though yes, tests sound like a good idea :)) while we figure out the LDS allocation issues.


http://reviews.llvm.org/D20728





More information about the llvm-commits mailing list