This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Remove old sample intrinsics
ClosedPublic

Authored by arsenm on Jan 22 2016, 2:00 AM.

Details

Summary

I did my best to try to update all the uses in tests that
just happened to use the old ones to the newer intrinsics.

I'm not sure I got all of the immediate operand conversions
correct, since the value seems to have been ignored by the
old pattern but I don't think it really matters.

Diff Detail

Event Timeline

arsenm updated this revision to Diff 45664.Jan 22 2016, 2:00 AM
arsenm retitled this revision from to AMDGPU: Remove old sample intrinsics.
arsenm updated this object.
arsenm added a reviewer: tstellarAMD.
arsenm added a subscriber: llvm-commits.
nhaehnle added inline comments.
lib/Target/AMDGPU/SIISelLowering.cpp
1369–1376

The method LowerSampleIntrinsic can now be removed, right?

arsenm updated this revision to Diff 45778.Jan 22 2016, 5:23 PM

Remove LowerSampleIntrinsic

nhaehnle accepted this revision.Jan 22 2016, 6:39 PM
nhaehnle added a reviewer: nhaehnle.

LGTM

This revision is now accepted and ready to land.Jan 22 2016, 6:39 PM
arsenm closed this revision.Jan 25 2016, 8:42 PM

r258787

test/CodeGen/AMDGPU/llvm.SI.sample.ll