Details
Diff Detail
- Repository
- rL LLVM
Event Timeline
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?
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?
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.