[PATCH] D20709: Support ARM subtarget feature +long64

Richard Smith via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 1 15:52:58 PDT 2016


rsmith added a subscriber: rsmith.
rsmith added a comment.

The summary and content of this patch are very different.

If you have an ABI variant that any language should be able to target, that should be controlled by the triple and related flags, not by a RenderScript language option.

If you want to introduce RenderScript as a supported language in Clang, that may well be reasonable, but that's a very different discussion.

Normally, we don't want to have ABI variants in clang that only a single language can target. There doesn't seem to be any fundamental reason why you couldn't compile C++ code for the RenderScript ABI, for instance, so tying the ABI to the language doesn't seem like the right approach.


http://reviews.llvm.org/D20709





More information about the cfe-commits mailing list