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

Jiangning Liu liujiangning1 at gmail.com
Mon Jul 7 23:34:13 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
>
>
>

- {F91790, layout=link}
- {F91789, layout=link}

http://reviews.llvm.org/D4361






More information about the llvm-commits mailing list