[PATCH] D117484: AMDGPU/GlobalISel: Handle legacy grid ID intrinsics

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

In D117484#3248451 <https://reviews.llvm.org/D117484#3248451>, @foad wrote:

> I don't understand. Why do we need to handle llvm.r600.* intrinsics on amdgcn subtargets? Why does globalisel need to handle them, when it doesn't support r600?

This is so old, it's a holdover when the backend was still called r600. Ideally clover would complete its migration to using an ABI that looks a lot closer to the HSA ABI, but that's been stuck for years. For the moment it's easier to keep the existing scheme working


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D117484/new/

https://reviews.llvm.org/D117484