[PATCH] Introduce arm_acle.h supporting existing LLVM builtin intrinsics

Saleem Abdulrasool compnerd at compnerd.org
Wed Jun 25 14:44:31 PDT 2014


Im sure that Im missing some context here.  I am not sure what using this wrapper header to map the intrinsics to a __builtin_ form really gains.  It adds an additional header that needs to be maintained, and requires that users include this header for the intrinsics.

At the very least, similar approaches to this on the Windows side have proven to be less than ideal.

Having some additional context around why this is preferable would be nice.

http://reviews.llvm.org/D4296






More information about the cfe-commits mailing list