[PATCH][AArch64]Implement vreinterpret ACLE functions in Clang
Hao Liu
Hao.Liu at arm.com
Sun Nov 17 05:30:19 PST 2013
Hi Chad,
Thanks for the review.
I've changed to use CHECK-LABEL and committed in
http://llvm.org/viewvc/llvm-project?view=revision&revision=194954
Thanks,
-Hao
-----Original Message-----
From: Chad Rosier [mailto:mcrosier at codeaurora.org]
Sent: Friday, November 15, 2013 2:17 AM
To: Hao Liu
Cc: cfe-commits at cs.uiuc.edu
Subject: Re: [PATCH][AArch64]Implement vreinterpret ACLE functions in Clang
Hao,
My only suggestion would be to use the CHECK-LABEL directive for each of the
function names. Otherwise, LGTM.
Chad
> Hi,
>
>
>
> The attached patch implements the vreinterpret functions in Clang. As
> such functions are implemented by typecast, there is no instruction
> generated in the backend. So there is no patch for LLVM.
>
>
>
> This patch is very simple, it only add new types and test cases.
>
> Review. Please.
>
>
>
> Thanks,
>
> -Hao
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by
The Linux Foundation
More information about the cfe-commits
mailing list