This is an archive of the discontinued LLVM Phabricator instance.

[OpenCL] AMDGPU: Add extensions cl_amd_media_ops and cl_amd_media_ops2
ClosedPublic

Authored by yaxunl on Aug 9 2016, 8:52 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

yaxunl updated this revision to Diff 67358.Aug 9 2016, 8:52 AM
yaxunl retitled this revision from to [OpenCL] AMDGPU: Add extensions cl_amd_media_ops and cl_amd_media_ops2.
yaxunl updated this object.
yaxunl added reviewers: nhaustov, Anastasia.
yaxunl added subscribers: cfe-commits, tstellarAMD.
Anastasia edited edge metadata.Aug 11 2016, 10:27 AM

Is this related to our discussion on cfe-dev about the extensions and also the earlier review you have created: https://reviews.llvm.org/D21698?

Is this related to our discussion on cfe-dev about the extensions and also the earlier review you have created: https://reviews.llvm.org/D21698?

Since that feature takes time to implement, we decide to add our extensions to Clang directly first.

Ok, sure. Is the plan to refactor this bit in case we implement the generic support later then?

It seems fine, although I can't check much without any documentation. Is there any reference available online?

Ok, sure. Is the plan to refactor this bit in case we implement the generic support later then?

It seems fine, although I can't check much without any documentation. Is there any reference available online?

Yes we will refactor it after the generic support is implemented.

The documentations are here:
https://www.khronos.org/registry/cl/extensions/amd/cl_amd_media_ops.txt
https://www.khronos.org/registry/cl/extensions/amd/cl_amd_media_ops2.txt

Do you think testing the declaration to be available without/with an error after the extension is enabled/disabled might be useful too?

Do you think testing the declaration to be available without/with an error after the extension is enabled/disabled might be useful too?

I feel it is not so useful for a specific extension.

When the feature which diagnoses functions disabled by extension is implemented, tests will be added as a generic case.

Anastasia accepted this revision.Aug 12 2016, 11:40 AM
Anastasia edited edge metadata.

LGTM!

This revision is now accepted and ready to land.Aug 12 2016, 11:40 AM
This revision was automatically updated to reflect the committed changes.