[PATCH] ARM & AArch64: share NEON builtin enumerators
t.p.northover at gmail.com
Thu Jan 30 03:58:53 PST 2014
Currently, both ARM and AArch64 targets make use of essentially the same __builtin_neon_XYZ intrinsics, which results in duplicated ARM::BI__builtin_neon_XYZ and AArch64::BI__builtin_neon_XYZ enumerators. Since the two targets share much of the NEON intrinsic codegen, these then have to be translated manually.
This patch factors out those enumerators into a separate "NEON" namespace. Currently, no further refactoring happens, but my intent is to (probably reasonably gradually) put ARM and AArch64 on a more equal footing where both sets of CodeGen defer to some kind of EmitNEONBuiltin function, instead of the current situation where AArch64 calls directly to EmitARMBuiltin.
The mapping between enumerator and builtin string is a little fragile, but I don't think significantly worse than before. Does the approach (and goal) look reasonable? And if so, can I commit this?
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 173504 bytes
Desc: not available
More information about the cfe-commits