[all-commits] [llvm/llvm-project] 56ffb8: [mlir] Stop allowing LLVMType Int arguments for GP...

Alexander Belyaev via All-commits all-commits at lists.llvm.org
Thu Sep 24 02:16:52 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 56ffb8d1697932c1097fba3315dba6d9e55b7bb8
      https://github.com/llvm/llvm-project/commit/56ffb8d1697932c1097fba3315dba6d9e55b7bb8
  Author: Alexander Belyaev <pifon at google.com>
  Date:   2020-09-24 (Thu, 24 Sep 2020)

  Changed paths:
    M mlir/include/mlir/Dialect/GPU/GPUOps.td
    M mlir/test/Conversion/GPUCommon/lower-launch-func-to-gpu-runtime-calls.mlir

  Log Message:
  -----------
  [mlir] Stop allowing LLVMType Int arguments for GPULaunchFuncOp.

Conversion to LLVM becomes confusing and incorrect if someone tries to lower
STD -> LLVM and only then GPULaunchFuncOp to LLVM separately. Although it is
technically allowed now, it works incorrectly because of the argument
promotion. The correct way to use this conversion pattern is to add to the
STD->LLVM patterns before running the pass.

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




More information about the All-commits mailing list