Re: [PATCH] D20709: For RenderScript, set alignment and width of long to 64-bits

2016-06-21 Thread Pirama Arumuga Nainar via cfe-commits
pirama abandoned this revision. pirama added a comment. Abandoning this revision. Patches for an alternative approach are listed above. http://reviews.llvm.org/D20709 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi

Re: [PATCH] D20709: For RenderScript, set alignment and width of long to 64-bits

2016-06-14 Thread Pirama Arumuga Nainar via cfe-commits
pirama added a comment. http://reviews.llvm.org/D21333 and http://reviews.llvm.org/D21334 add the RenderScript triple and modify 'long' based off the triple. r272317, r272342, r272438 take care of the LangOpt and handling of '-x renderscript' and '.rs' extensions. http://reviews.llvm.org/D20

Re: [PATCH] D20709: For RenderScript, set alignment and width of long to 64-bits

2016-06-07 Thread Richard Smith via cfe-commits
rsmith added a comment. In http://reviews.llvm.org/D20709#446595, @srhines wrote: > There is a coming patch that adds a single attribute (for "kernel"), that has > error checking to ensure that it is only used by our RenderScript targets. I > assume that the cleanest way to retain that diagnost

Re: [PATCH] D20709: For RenderScript, set alignment and width of long to 64-bits

2016-06-01 Thread Stephen Hines via cfe-commits
srhines added a comment. After talking to Richard this afternoon on IRC, we are going with a slightly different plan here. We will create two new triples (renderscript32-*-* and renderscript64-*-*), which are direct subclasses of the general Android ARM32 and ARM64 targets. In this case, we can