[PATCH][AArch64] implement aarch64 neon instruction class AdvSIMD (3 diff)

Jiangning Liu liujiangning1 at gmail.com
Sun Sep 1 23:12:40 PDT 2013


Hi Tim,

Here come new patches for review.

1) Changed some naming conventions in both clang and llvm patches. For
example, change _High to _Hi in clang, and change _8H to _8h8b in .td file
of llvm.
2) Change all addw and subw to be not commutable
3) Modified pattern match for instructions relevant to the followings,

ARM::BI__builtin_neon_vaddhn_v
ARM::BI__builtin_neon_vsubhn_v
ARM::BI__builtin_neon_vqdmlal_v
ARM::BI__builtin_neon_vqdmlsl_v

4) Defined some more PatFrag to simplify the patterns being matched.
5) Some test cases are also modified accordingly.

Thanks,
-Jiangning


2013/8/27 Tim Northover <tnorthover at apple.com>

> > I think essentially you are intending to remove some of the builtins
> from 32-bit ARM target.
>
> Yep, that's correct. And your list matches with my plans for now too.
>
> > Do you mean to use lshr instead of shl, because addhn is to get high
> narrow, so we could have,
>
> Yes, I did sorry. I'd discovered this, and was planning to mention it when
> posting the patches.
>
> Cheers.
>
> Tim.
>



-- 
Thanks,
-Jiangning
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130902/279ee3e6/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: clang-3vdiff-1.patch
Type: application/octet-stream
Size: 49251 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130902/279ee3e6/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: llvm-3vdiff-1.patch
Type: application/octet-stream
Size: 169767 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130902/279ee3e6/attachment-0001.obj>


More information about the cfe-commits mailing list