BPF: misc minor review fixes
This commit is contained in:
parent
bd8e5ce4b9
commit
ab93a139ef
5 changed files with 11 additions and 11 deletions
|
@ -12,6 +12,10 @@ pub fn opts(endian: Endian) -> TargetOptions {
|
|||
no_builtins: true,
|
||||
panic_strategy: PanicStrategy::Abort,
|
||||
position_independent_executables: true,
|
||||
// Disable MergeFunctions since:
|
||||
// - older kernels don't support bpf-to-bpf calls
|
||||
// - on newer kernels, userspace still needs to relocate before calling
|
||||
// BPF_PROG_LOAD and not all BPF libraries do that yet
|
||||
merge_functions: MergeFunctions::Disabled,
|
||||
obj_is_bitcode: true,
|
||||
requires_lto: false,
|
||||
|
|
|
@ -235,8 +235,8 @@ impl Step for Llvm {
|
|||
let llvm_targets = match &builder.config.llvm_targets {
|
||||
Some(s) => s,
|
||||
None => {
|
||||
"AArch64;ARM;Hexagon;MSP430;Mips;NVPTX;PowerPC;RISCV;\
|
||||
Sparc;SystemZ;WebAssembly;X86;BPF"
|
||||
"AArch64;ARM;BPF;Hexagon;MSP430;Mips;NVPTX;PowerPC;RISCV;\
|
||||
Sparc;SystemZ;WebAssembly;X86"
|
||||
}
|
||||
};
|
||||
|
||||
|
|
|
@ -2311,10 +2311,6 @@ impl<'test> TestCx<'test> {
|
|||
// No extra flags needed.
|
||||
}
|
||||
|
||||
Some("bpf-linker") => {
|
||||
rustc.arg("-Clink-args=--emit=asm");
|
||||
}
|
||||
|
||||
Some(_) => self.fatal("unknown 'assembly-output' header"),
|
||||
None => self.fatal("missing 'assembly-output' header"),
|
||||
}
|
||||
|
|
|
@ -48,8 +48,8 @@ const ARCH_TABLE: &[(&str, &str)] = &[
|
|||
("armv7s", "arm"),
|
||||
("asmjs", "asmjs"),
|
||||
("avr", "avr"),
|
||||
("bpfeb", "bpfeb"),
|
||||
("bpfel", "bpfel"),
|
||||
("bpfeb", "bpf"),
|
||||
("bpfel", "bpf"),
|
||||
("hexagon", "hexagon"),
|
||||
("i386", "x86"),
|
||||
("i586", "x86"),
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue