Re: [PATCH] D21198: RenderScript support in the Frontend

2016-06-09 Thread Pirama Arumuga Nainar via cfe-commits
pirama added a comment. Thanks for the review. Docs update is in http://reviews.llvm.org/D21212 Repository: rL LLVM http://reviews.llvm.org/D21198 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/list

Re: [PATCH] D21198: RenderScript support in the Frontend

2016-06-09 Thread Pirama Arumuga Nainar via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL272342: RenderScript support in the Frontend (authored by pirama). Changed prior to commit: http://reviews.llvm.org/D21198?vs=60277&id=60279#toc Repository: rL LLVM http://reviews.llvm.org/D21198 F

Re: [PATCH] D21198: RenderScript support in the Frontend

2016-06-09 Thread Pirama Arumuga Nainar via cfe-commits
pirama updated this revision to Diff 60277. pirama added a comment. Reorder IK_RenderScript before IK_AST like Richard had requested. http://reviews.llvm.org/D21198 Files: include/clang/Basic/Attr.td include/clang/Basic/LangOptions.def include/clang/Frontend/FrontendOptions.h lib/Fronte

Re: [PATCH] D21198: RenderScript support in the Frontend

2016-06-09 Thread Pirama Arumuga Nainar via cfe-commits
pirama updated this revision to Diff 60252. pirama added a comment. Cleanup bad merge http://reviews.llvm.org/D21198 Files: include/clang/Basic/Attr.td include/clang/Basic/LangOptions.def include/clang/Frontend/FrontendOptions.h lib/Frontend/CompilerInvocation.cpp lib/Frontend/Fronten

Re: [PATCH] D21198: RenderScript support in the Frontend

2016-06-09 Thread Pirama Arumuga Nainar via cfe-commits
pirama added a comment. Oops, this update merged changes from http://reviews.llvm.org/D21199 as well. Let me clean up and upload a new patch. http://reviews.llvm.org/D21198 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.

Re: [PATCH] D21198: RenderScript support in the Frontend

2016-06-09 Thread Pirama Arumuga Nainar via cfe-commits
pirama updated this revision to Diff 60247. pirama updated the summary for this revision. pirama added a comment. Re-ordered enum http://reviews.llvm.org/D21198 Files: include/clang/Basic/Attr.td include/clang/Basic/LangOptions.def include/clang/Driver/Types.def include/clang/Frontend/F

Re: [PATCH] D21198: RenderScript support in the Frontend

2016-06-09 Thread Richard Smith via cfe-commits
rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. 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 maintainer