Remove obsolete ARM intrinsics vclz and vcnt

Steven Wu stevenwu at apple.com
Wed Nov 5 08:40:09 PST 2014


Hi Renato

These intrinsics have been auto upgrade for a while and there are tests for auto upgrade of these ARM intrinsics (test/Bitcode/arm32_neon_vcnt_upgrade.ll). This patch is an API cleanup to prevent generating these old ARM intrinsics.

There are no tests broken after this change and we are still compatible with these two intrinsics.

Thanks

Steven
> On Nov 5, 2014, at 1:43 AM, Renato Golin <renato.golin at linaro.org> wrote:
> 
> Hi Steven,
> 
> Aren't there *any* tests for that that should now fail? Shouldn't we
> change the ARM tests to check for the new intrinsic?
> 
> cheers,
> --renato
> 
> On 4 November 2014 22:14, Steven Wu <stevenwu at apple.com> wrote:
>> This patch removes llvm.arm.neon.vclz and llvm.arm.neon.vcnt. Both intrinsics will be upgraded through AutoUpgrade to target independent intrinsics.
>> 
>> We shouldn’t support the attempt to upgrade to them and we have no reason to keep these two intrinsics around.
>> 
>> Steven
>> 
>> 
>> 
>> 
>> 
>> 
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>> 





More information about the llvm-commits mailing list