This is an archive of the discontinued LLVM Phabricator instance.

RenderScript support in the Frontend
ClosedPublic

Authored by pirama on Jun 9 2016, 2:24 PM.

Details

Summary

Create a new Frontend LangOpt to specify the renderscript language. It is enabled by the "-x renderscript" option from the driver.

Add a "kernel" function attribute only for RenderScript (an "ignored attribute" warning is generated otherwise).

Make the NativeHalfType and NativeHalfArgsAndReturns LangOpts be implied by the RenderScript LangOpt.

Diff Detail

Repository
rL LLVM

Event Timeline

pirama updated this revision to Diff 60236.Jun 9 2016, 2:24 PM
pirama retitled this revision from to Add a RenderScript language type.
pirama updated this object.
pirama added a reviewer: rsmith.
pirama added subscribers: srhines, cfe-commits.
pirama retitled this revision from Add a RenderScript language type to RenderScript support in the Frontend.Jun 9 2016, 2:26 PM
pirama updated this object.
pirama updated this object.
rsmith accepted this revision.Jun 9 2016, 2:58 PM
rsmith edited edge metadata.

Please make sure that any relevant parts of our documentation are also updated. Given how small and self-contained this is, that we have a reasonable expectation of good support and maintainership, and that it's a reasonably well-established language variant, I don't have any problem with it living in upstream clang.

include/clang/Frontend/FrontendOptions.h
77–79 ↗(On Diff #60236)

Please reorder this before IK_AST, so that we keep the source languages before the somewhat-more clang-internal things.

This revision is now accepted and ready to land.Jun 9 2016, 2:58 PM
pirama updated this revision to Diff 60247.Jun 9 2016, 3:03 PM
pirama updated this object.
pirama edited edge metadata.

Re-ordered enum

pirama added a comment.Jun 9 2016, 3:06 PM

Oops, this update merged changes from D21199 as well. Let me clean up and upload a new patch.

pirama updated this revision to Diff 60252.Jun 9 2016, 3:08 PM

Cleanup bad merge

pirama updated this revision to Diff 60277.Jun 9 2016, 4:40 PM

Reorder IK_RenderScript before IK_AST like Richard had requested.

This revision was automatically updated to reflect the committed changes.
pirama added a comment.Jun 9 2016, 4:43 PM

Thanks for the review. Docs update is in D21212