[PATCH] D46315: [RegUsageInfoCollector] Fix handling of callee saved registers with CSR optimization.

Quentin Colombet via llvm-commits llvm-commits at lists.llvm.org
Wed May 23 13:09:09 PDT 2018


> Doesn't the target already do this?  At least there's a CoveredBySubRegs flag in the Register .td class ...

Indeed! Forgot about that one too!

2018-05-19 5:13 GMT-07:00 Ulrich Weigand via Phabricator
<reviews at reviews.llvm.org>:
> uweigand added a comment.
>
> In https://reviews.llvm.org/D46315#1105144, @qcolombet wrote:
>
>> An alternative of proving that, would be to have the target explicitly say whether or not their register units covers the full registers.
>
>
> Doesn't the target already do this?  At least there's a CoveredBySubRegs flag in the Register .td class ...
>
>
> https://reviews.llvm.org/D46315
>
>
>


More information about the llvm-commits mailing list