[PATCH] Implement aarch64 neon instruction class AdvSIMD (by element) - Clang
apazos at codeaurora.org
Tue Oct 1 10:02:24 PDT 2013
In the clang changes, some of the intrinsics name carry a 'q' at the end of the base function name ( e..g., vmla_laneq). As a consequence you have to define a Q and non-Q version of the intrinsic declaration.
Do you think it is better to add the 'q' using a size modifier like "128b but mangling 'q' at the end of base neon function name". I did something like that for the scalar by element intrinsics where I need to generate names like 'vmulxd_laneq_f64'.
But in your case it seems you need 'q' both before the first '_' and at the end of the base neon function name. Is that the reason you did not use a type modifier? You would need yet another size modifier like '128b but mangling 'q' at both places".
It would be best if we agree on a common way to handle the 'q' at the end of base neon function name.
From: cfe-commits-bounces at cs.uiuc.edu [mailto:cfe-commits-bounces at cs.uiuc.edu] On Behalf Of Jiangning Liu
Sent: Tuesday, October 01, 2013 6:42 AM
To: t.p.northover at gmail.com; liujiangning1 at gmail.com
Cc: llvm-commits at cs.uiuc.edu; cfe-commits at cs.uiuc.edu
Subject: Re: [PATCH] Implement aarch64 neon instruction class AdvSIMD (by element) - Clang
CHANGE SINCE LAST DIFF
More information about the llvm-commits