[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 23:19:14 PDT 2018


MaskRay added a comment.

In https://reviews.llvm.org/D52759#1251949, @arsenm wrote:

> Why do other targets do this? -debug already behaves weirdly since -debug prints more during SelectionDAG than -debug-only=isel, and this will make that worse?


Other targets don't have a `*-isel` pass. I am unclear why AMDGPU has the `isel` that misguided people to do things like `-stop-after=isel` (which does not work if they don't build the AMDGPU target)


Repository:
  rL LLVM

https://reviews.llvm.org/D52759





More information about the llvm-commits mailing list