[all-commits] [llvm/llvm-project] e75ce7: [amdgpu][lds] Fix missing markUsedByKernel calls a...

Jon Chesterfield via All-commits all-commits at lists.llvm.org
Tue Jul 11 16:37:45 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e75ce77cd7a7ebea5f26dc58048b1f0b9daddb74
      https://github.com/llvm/llvm-project/commit/e75ce77cd7a7ebea5f26dc58048b1f0b9daddb74
  Author: Jon Chesterfield <jonathanchesterfield at gmail.com>
  Date:   2023-07-12 (Wed, 12 Jul 2023)

  Changed paths:
    M llvm/lib/Target/AMDGPU/AMDGPULowerModuleLDSPass.cpp
    A llvm/test/CodeGen/AMDGPU/lower-module-lds-all-indirect-accesses.ll
    M llvm/test/CodeGen/AMDGPU/lower-module-lds-offsets.ll
    M llvm/test/CodeGen/AMDGPU/lower-module-lds-single-var-unambiguous.ll
    M llvm/test/CodeGen/AMDGPU/lower-module-lds-via-hybrid.ll
    M llvm/test/CodeGen/AMDGPU/lower-module-lds-via-table.ll

  Log Message:
  -----------
  [amdgpu][lds] Fix missing markUsedByKernel calls and undef lookup table elements

More robust association between the kernels and lds struct.

Use poison instead of value() for lookup table elements introduced by dynamic lds lowering.

Extracted from D154946, new test from there verbatim. Segv fixed.

Fixes issues/63338

Fixes SWDEV-404491

Reviewed By: arsenm

Differential Revision: https://reviews.llvm.org/D154972




More information about the All-commits mailing list