Skip to content

Commit

Permalink
x64 new backend: port ABI implementation to shared infrastructure wit…
Browse files Browse the repository at this point in the history
…h AArch64.

Previously, in #2128, we factored out a common "vanilla 64-bit ABI"
implementation from the AArch64 ABI code, with the idea that this should
be largely compatible with x64. This PR alters the new x64 backend to
make use of the shared infrastructure, removing the duplication that
existed previously. The generated code is nearly (not exactly) the same;
the only difference relates to how the clobber-save region is padded in
the prologue.

This also changes some register allocations in the aarch64 code because
call support in the shared ABI infra now passes a temp vreg in, rather
than requiring use of a fixed, non-allocable temp; tests have been
updated, and the runtime behavior is unchanged.
  • Loading branch information
cfallin committed Aug 18, 2020
1 parent 7fdbd08 commit 719705b
Show file tree
Hide file tree
Showing 6 changed files with 588 additions and 1,141 deletions.
5 changes: 3 additions & 2 deletions cranelift/codegen/src/isa/aarch64/abi.rs
Original file line number Diff line number Diff line change
Expand Up @@ -544,6 +544,7 @@ impl ABIMachineImpl for AArch64MachineImpl {
defs: Vec<Writable<Reg>>,
loc: SourceLoc,
opcode: ir::Opcode,
tmp: Writable<Reg>,
) -> SmallVec<[(/* is_safepoint = */ bool, Inst); 2]> {
let mut insts = SmallVec::new();
match &dest {
Expand All @@ -563,7 +564,7 @@ impl ABIMachineImpl for AArch64MachineImpl {
insts.push((
false,
Inst::LoadExtName {
rd: writable_spilltmp_reg(),
rd: tmp,
name: Box::new(name.clone()),
offset: 0,
srcloc: loc,
Expand All @@ -573,7 +574,7 @@ impl ABIMachineImpl for AArch64MachineImpl {
true,
Inst::CallInd {
info: Box::new(CallIndInfo {
rn: spilltmp_reg(),
rn: tmp.to_reg(),
uses,
defs,
loc,
Expand Down
Loading

0 comments on commit 719705b

Please sign in to comment.