mirror of
https://github.com/rust-lang/rust.git
synced 2024-11-27 01:04:03 +00:00
897c7bb23b
Similar to prior support added for the mips430, avr, and x86 targets
this change implements the rough equivalent of clang's
[`__attribute__((interrupt))`][clang-attr] for riscv targets, enabling
e.g.
```rust
static mut CNT: usize = 0;
pub extern "riscv-interrupt-m" fn isr_m() {
unsafe {
CNT += 1;
}
}
```
to produce highly effective assembly like:
```asm
pub extern "riscv-interrupt-m" fn isr_m() {
420003a0: 1141 addi sp,sp,-16
unsafe {
CNT += 1;
420003a2: c62a sw a0,12(sp)
420003a4: c42e sw a1,8(sp)
420003a6: 3fc80537 lui a0,0x3fc80
420003aa: 63c52583 lw a1,1596(a0) # 3fc8063c <_ZN12esp_riscv_rt3CNT17hcec3e3a214887d53E.0>
420003ae: 0585 addi a1,a1,1
420003b0: 62b52e23 sw a1,1596(a0)
}
}
420003b4: 4532 lw a0,12(sp)
420003b6: 45a2 lw a1,8(sp)
420003b8: 0141 addi sp,sp,16
420003ba: 30200073 mret
```
(disassembly via `riscv64-unknown-elf-objdump -C -S --disassemble ./esp32c3-hal/target/riscv32imc-unknown-none-elf/release/examples/gpio_interrupt`)
This outcome is superior to hand-coded interrupt routines which, lacking
visibility into any non-assembly body of the interrupt handler, have to
be very conservative and save the [entire CPU state to the stack
frame][full-frame-save]. By instead asking LLVM to only save the
registers that it uses, we defer the decision to the tool with the best
context: it can more accurately account for the cost of spills if it
knows that every additional register used is already at the cost of an
implicit spill.
At the LLVM level, this is apparently [implemented by] marking every
register as "[callee-save]," matching the semantics of an interrupt
handler nicely (it has to leave the CPU state just as it found it after
its `{m|s}ret`).
This approach is not suitable for every interrupt handler, as it makes
no attempt to e.g. save the state in a user-accessible stack frame. For
a full discussion of those challenges and tradeoffs, please refer to
[the interrupt calling conventions RFC][rfc].
Inside rustc, this implementation differs from prior art because LLVM
does not expose the "all-saved" function flavor as a calling convention
directly, instead preferring to use an attribute that allows for
differentiating between "machine-mode" and "superivsor-mode" interrupts.
Finally, some effort has been made to guide those who may not yet be
aware of the differences between machine-mode and supervisor-mode
interrupts as to why no `riscv-interrupt` calling convention is exposed
through rustc, and similarly for why `riscv-interrupt-u` makes no
appearance (as it would complicate future LLVM upgrades).
[clang-attr]: https://clang.llvm.org/docs/AttributeReference.html#interrupt-risc-v
[full-frame-save]: 9281af2ecf/src/lib.rs (L440-L469)
[implemented by]: b7fb2a3fec/llvm/lib/Target/RISCV/RISCVRegisterInfo.cpp (L61-L67)
[callee-save]: 973f1fe7a8/llvm/lib/Target/RISCV/RISCVCallingConv.td (L30-L37)
[rfc]: https://github.com/rust-lang/rfcs/pull/3246
62 lines
2.1 KiB
Plaintext
62 lines
2.1 KiB
Plaintext
error[E0570]: `"ptx-kernel"` is not a supported ABI for the current target
|
|
--> $DIR/unsupported.rs:30:1
|
|
|
|
|
LL | extern "ptx-kernel" fn ptx() {}
|
|
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
|
|
|
error[E0570]: `"amdgpu-kernel"` is not a supported ABI for the current target
|
|
--> $DIR/unsupported.rs:32:1
|
|
|
|
|
LL | extern "amdgpu-kernel" fn amdgpu() {}
|
|
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
|
|
|
error[E0570]: `"wasm"` is not a supported ABI for the current target
|
|
--> $DIR/unsupported.rs:34:1
|
|
|
|
|
LL | extern "wasm" fn wasm() {}
|
|
| ^^^^^^^^^^^^^^^^^^^^^^^
|
|
|
|
error[E0570]: `"msp430-interrupt"` is not a supported ABI for the current target
|
|
--> $DIR/unsupported.rs:42:1
|
|
|
|
|
LL | extern "msp430-interrupt" fn msp430() {}
|
|
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
|
|
|
error[E0570]: `"avr-interrupt"` is not a supported ABI for the current target
|
|
--> $DIR/unsupported.rs:44:1
|
|
|
|
|
LL | extern "avr-interrupt" fn avr() {}
|
|
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
|
|
|
error[E0570]: `"riscv-interrupt-m"` is not a supported ABI for the current target
|
|
--> $DIR/unsupported.rs:46:1
|
|
|
|
|
LL | extern "riscv-interrupt-m" fn riscv() {}
|
|
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
|
|
|
error[E0570]: `"x86-interrupt"` is not a supported ABI for the current target
|
|
--> $DIR/unsupported.rs:51:1
|
|
|
|
|
LL | extern "x86-interrupt" fn x86() {}
|
|
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
|
|
|
error[E0570]: `"thiscall"` is not a supported ABI for the current target
|
|
--> $DIR/unsupported.rs:56:1
|
|
|
|
|
LL | extern "thiscall" fn thiscall() {}
|
|
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
|
|
|
warning: use of calling convention not supported on this target
|
|
--> $DIR/unsupported.rs:62:1
|
|
|
|
|
LL | extern "stdcall" fn stdcall() {}
|
|
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
|
|
|
|
= warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release!
|
|
= note: for more information, see issue #87678 <https://github.com/rust-lang/rust/issues/87678>
|
|
= note: `#[warn(unsupported_calling_conventions)]` on by default
|
|
|
|
error: aborting due to 8 previous errors; 1 warning emitted
|
|
|
|
For more information about this error, try `rustc --explain E0570`.
|