[PATCH] [AArch64] Enhance rematerialization by adding a new API isAsCheapAsAMove in TargetInstroInfo

Jiangning Liu liujiangning1 at gmail.com
Mon Jul 7 23:34:09 PDT 2014


Hi Eric,

OK. Attached are two separate patches.

Hi Qunentin,

I modified the code following your new comments.

Since I don't know how phabaricator will work for two patches uploaded, I'm
simply reply this mail by attached two separate patches. I don't want to
create a new review entry in phabaricator to confuse the people who want to
review the patch.

Thanks,
-Jiangning


2014-07-08 5:25 GMT+08:00 Eric Christopher <echristo at gmail.com>:

> You could also split the patch into adding the API and using it in the
> next commit. I.e. there's no reason why isRematerializable can't be set for
> the instructions separate from adding TII->isCheapAsAMove()
>
> http://reviews.llvm.org/D4361
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140708/9313c5e9/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Add-TargetInstrInfo-interface-isAsCheapAsAMove.patch
Type: text/x-patch
Size: 3852 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140708/9313c5e9/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Implement-AArch64-TTI-interface-isAsCheapAsAMove.patch
Type: text/x-patch
Size: 5971 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140708/9313c5e9/attachment-0001.bin>


More information about the llvm-commits mailing list