Skip to content

Commit 93ef808

Browse files
committed
Revert vita's c_char back to i8
1 parent 942db67 commit 93ef808

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

library/core/src/ffi/primitives.rs

+6-2
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,6 @@ mod c_char_definition {
3939
// These are the targets on which c_char is unsigned. Usually the
4040
// signedness is the same for all target_os values on a given architecture
4141
// but there are some exceptions (see isSignedCharDefault() in clang).
42-
//
4342
// aarch64:
4443
// Section 10 "Arm C and C++ language mappings" in Procedure Call Standard for the Arm®
4544
// 64-bit Architecture (AArch64) says C/C++ char is unsigned byte.
@@ -97,14 +96,19 @@ mod c_char_definition {
9796
// are promoted to int as if from type signed char by default, unless the /J compilation
9897
// option is used."
9998
// https://learn.microsoft.com/en-us/cpp/cpp/fundamental-types-cpp?view=msvc-170#character-types
99+
// Vita:
100+
// Chars are signed by default on the Vita, and VITASDK follows that convention.
101+
// https://github.com/vitasdk/buildscripts/blob/09c533b771591ecde88864b6acad28ffb688dbd4/patches/gcc/0001-gcc-10.patch#L33-L34
102+
//
100103
// L4Re:
101-
// The kernel builds with -funsigned-char on all targets (but useserspace follows the
104+
// The kernel builds with -funsigned-char on all targets (but userspace follows the
102105
// architecture defaults). As we only have a target for userspace apps so there are no
103106
// special cases for L4Re below.
104107
// https://github.com/rust-lang/rust/pull/132975#issuecomment-2484645240
105108
if #[cfg(all(
106109
not(windows),
107110
not(target_vendor = "apple"),
111+
not(target_os = "vita"),
108112
any(
109113
target_arch = "aarch64",
110114
target_arch = "arm",

0 commit comments

Comments
 (0)