[PATCH] D52759: [AMDGPU] Rename pass "isel" to amdgpu-isel

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 1 17:37:48 PDT 2018


MaskRay created this revision.
MaskRay added reviewers: tstellar, echristo.
Herald added subscribers: llvm-commits, kristof.beyls, t-tye, tpr, dstuttard, yaxunl, nhaehnle, wdng, jvesely, kzhuravl, arsenm.
Herald added a reviewer: javed.absar.

This matches what other targets do, e.g.  x86-isel aarch64-isel


Repository:
  rL LLVM

https://reviews.llvm.org/D52759

Files:
  lib/Target/AMDGPU/AMDGPUISelDAGToDAG.cpp
  test/CodeGen/AMDGPU/legalize-fp-load-invariant.ll
  test/CodeGen/AMDGPU/print-mir-custom-pseudo.ll


Index: test/CodeGen/AMDGPU/print-mir-custom-pseudo.ll
===================================================================
--- test/CodeGen/AMDGPU/print-mir-custom-pseudo.ll
+++ test/CodeGen/AMDGPU/print-mir-custom-pseudo.ll
@@ -1,4 +1,4 @@
-; RUN: llc -mtriple=amdgcn--amdpal -mcpu=fiji -print-after=isel -verify-machineinstrs < %s 2>&1 | FileCheck -check-prefix=SI %s
+; RUN: llc -mtriple=amdgcn--amdpal -mcpu=fiji -print-after=amdgpu-isel -verify-machineinstrs < %s 2>&1 | FileCheck -check-prefix=SI %s
 
 ; This checks that the -print-after of MIR containing a target custom pseudo
 ; value works correctly.
Index: test/CodeGen/AMDGPU/legalize-fp-load-invariant.ll
===================================================================
--- test/CodeGen/AMDGPU/legalize-fp-load-invariant.ll
+++ test/CodeGen/AMDGPU/legalize-fp-load-invariant.ll
@@ -1,4 +1,4 @@
-; RUN: llc -march=amdgcn -mcpu=tahiti -verify-machineinstrs -stop-after=isel -o - %s | FileCheck -check-prefix=GCN %s
+; RUN: llc -march=amdgcn -mcpu=tahiti -verify-machineinstrs -stop-after=amdgpu-isel -o - %s | FileCheck -check-prefix=GCN %s
 
 ; Type legalization for illegal FP type results was dropping invariant
 ; and dereferenceable flags.
Index: lib/Target/AMDGPU/AMDGPUISelDAGToDAG.cpp
===================================================================
--- lib/Target/AMDGPU/AMDGPUISelDAGToDAG.cpp
+++ lib/Target/AMDGPU/AMDGPUISelDAGToDAG.cpp
@@ -242,12 +242,12 @@
 
 }  // end anonymous namespace
 
-INITIALIZE_PASS_BEGIN(AMDGPUDAGToDAGISel, "isel",
+INITIALIZE_PASS_BEGIN(AMDGPUDAGToDAGISel, "amdgpu-isel",
                       "AMDGPU DAG->DAG Pattern Instruction Selection", false, false)
 INITIALIZE_PASS_DEPENDENCY(AMDGPUArgumentUsageInfo)
 INITIALIZE_PASS_DEPENDENCY(AMDGPUPerfHintAnalysis)
 INITIALIZE_PASS_DEPENDENCY(LegacyDivergenceAnalysis)
-INITIALIZE_PASS_END(AMDGPUDAGToDAGISel, "isel",
+INITIALIZE_PASS_END(AMDGPUDAGToDAGISel, "amdgpu-isel",
                     "AMDGPU DAG->DAG Pattern Instruction Selection", false, false)
 
 /// This pass converts a legalized DAG into a AMDGPU-specific


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D52759.167865.patch
Type: text/x-patch
Size: 2080 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181002/dbf6acd9/attachment.bin>


More information about the llvm-commits mailing list