[all-commits] [llvm/llvm-project] cab44c: [mlir][AMDGPU] Add --chipset option to AMDGPUToROCDL

Krzysztof Drewniak via All-commits all-commits at lists.llvm.org
Thu Jul 7 07:58:30 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: cab44c515c63db876a7f626fdc36be4a9157ebad
      https://github.com/llvm/llvm-project/commit/cab44c515c63db876a7f626fdc36be4a9157ebad
  Author: Krzysztof Drewniak <Krzysztof.Drewniak at amd.com>
  Date:   2022-07-07 (Thu, 07 Jul 2022)

  Changed paths:
    M mlir/include/mlir/Conversion/AMDGPUToROCDL/AMDGPUToROCDL.h
    A mlir/include/mlir/Conversion/AMDGPUToROCDL/Chipset.h
    M mlir/include/mlir/Conversion/GPUToROCDL/GPUToROCDLPass.h
    M mlir/include/mlir/Conversion/Passes.td
    M mlir/include/mlir/Dialect/AMDGPU/AMDGPU.td
    M mlir/lib/Conversion/AMDGPUToROCDL/AMDGPUToROCDL.cpp
    M mlir/lib/Conversion/AMDGPUToROCDL/CMakeLists.txt
    A mlir/lib/Conversion/AMDGPUToROCDL/Chipset.cpp
    M mlir/lib/Conversion/GPUToROCDL/LowerGpuOpsToROCDLOps.cpp
    M mlir/test/Conversion/AMDGPUToROCDL/amdgpu-to-rocdl.mlir
    M mlir/test/Dialect/AMDGPU/ops.mlir

  Log Message:
  -----------
  [mlir][AMDGPU] Add --chipset option to AMDGPUToROCDL

Because the buffer descriptor structure (the V#) has no backwards-compatibility
guarentees, and since said guarantees have been violated in practice
(see https://github.com/llvm/llvm-project/issues/56323 ), and since
the `targetIsRDNA` attribute isn't something that higher-level clients can set
in general, make the lowering of the amdgpu dialect to rocdl take a --chipset
option.

Note that this option is a string because adding a parser for the Chipset
struct to llvm::cl wasn't working out.

Reviewed By: herhut

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




More information about the All-commits mailing list