[PATCH] D62686: [RISCV] Add support for save/restore of callee-saved registers via libcalls

[Lewis Revill via Phabricator via llvm-commits]

lewis-revill added a comment.
Herald added a subscriber: s.egerton.

In D62686#1571192 <https://reviews.llvm.org/D62686#1571192>, @asb wrote:

> Hi Lewis, did you evaluate implementing the machine outliner as a more general approach for getting code size savings in this sort of way?
>
> You should have tests for at least a subset of target-abi {ilp32,lp64}{f,d} and the compiler shouldn't assert in those cases.


Would the existence of save/restore in the codebase conflict with the machine outliner approach? Otherwise it should be a case of simply disallowing enabling both flags at once if this leads to invalid behaviour.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62686/new/

https://reviews.llvm.org/D62686