This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: rename OpenCL lowering pass to be R600 specific.
ClosedPublic

Authored by airlied on May 1 2018, 8:15 PM.

Details

Reviewers
jvesely
Summary
This pass is
  a) broken.
  b) r600 specific.
  
  Fixing (a) is a bit more non-trivial, but fixing (b)
  is easy. Move this pass to being R600 only for now.
  
  This pass does pass all the unit tests, however clang
  no longer generates code that looks like the unit test
  input, so fixing the pass requires fixing the tests and
  the pass as one, and checking it works with clang still.

Diff Detail

Repository
rL LLVM

Event Timeline

airlied created this revision.May 1 2018, 8:15 PM
jvesely accepted this revision.May 2 2018, 4:46 PM
jvesely added a subscriber: jvesely.

LGTM

This revision is now accepted and ready to land.May 2 2018, 4:46 PM
arsenm closed this revision.May 13 2018, 3:09 AM

r332196