about summary refs log tree commit diff
path: root/compiler/rustc_codegen_cranelift/src/inline_asm.rs
diff options
context:
space:
mode:
authorbors <bors@rust-lang.org>2024-11-09 04:43:51 +0000
committerbors <bors@rust-lang.org>2024-11-09 04:43:51 +0000
commit012ae13d6ac41fb4df649d39979f39b08241a290 (patch)
tree3cf818f981b903a1beadb263e912f6ad29f8e803 /compiler/rustc_codegen_cranelift/src/inline_asm.rs
parent328b759142ddeae96da83176f103200009d3e3f1 (diff)
parent89d7efaf8f5ed8c1c15faea21824ae479656bdc1 (diff)
downloadrust-012ae13d6ac41fb4df649d39979f39b08241a290.tar.gz
rust-012ae13d6ac41fb4df649d39979f39b08241a290.zip
Auto merge of #132549 - Zalathar:rust-string, r=cuviper
Make `RustString` an extern type to avoid `improper_ctypes` warnings

Currently, any FFI function that uses `&RustString` needs to also add `#[ignore(improper_ctypes)]` to silence a warning.

The warning is not _completely_ bogus, because `RustString` contains `Vec<u8>` and therefore does not have a guaranteed layout. But we have no way of telling the lint that this doesn't matter, because the C++ code only uses that pointer opaquely and never relies on its underlying layout.

Ideally there would be some way to silence `improper_ctypes` at the type-definition site. But because there isn't, casting to and from a separate extern type is better than having to annotate every single use site.
Diffstat (limited to 'compiler/rustc_codegen_cranelift/src/inline_asm.rs')
0 files changed, 0 insertions, 0 deletions