Repository navigation
std: Combine is_mips64_feature_detected into is_mips_feature_detected and enable on MIPS release 6 - #165007
std: Combine is_mips64_feature_detected into is_mips_feature_detected and enable on MIPS release 6#165007tgross35 wants to merge 2 commits into
is_mips64_feature_detected into is_mips_feature_detected and enable on MIPS release 6#165007Conversation
We have four MIPS-like arches, not just the two.
…ted` All other architecture families currently use a single macro for both 32- and 64-bit versions. Do the same for MIPS.
|
cc @Amanieu, @folkertdev, @sayantn Some changes occurred in cc @Amanieu, @folkertdev, @sayantn |
|
Mips maintainers @Gelbpunkt @LukasWoodtli @chenx97 @709924470 @Cyanoxygen @Fearyncess could you confirm that this makes sense? The macro currently isn't very useful, but as long as most of the features are the same across the four arch variants then we may as well do the same thing as other arches. |
|
I believe this makes sense, yes. Most target features that may be listed here in the future should apply to both 32-bit and 64-bit MIPS. I believe |
| "loongarch32" | "loongarch64" => "is_loongarch_feature_detected", | ||
| "s390x" => "is_s390x_feature_detected", | ||
| "mips64" | "mips64el" => "is_mips64_feature_detected", | ||
| "mips" | "mips32r6" | "mips64" | "mips64el" => "is_mips_feature_detected", |
There was a problem hiding this comment.
Get ready for some target triple madness
mips mipsel mips64 mips64el mipsisa32r6 mipsisa32r6el mipsisa64r6 mipsisa64r6el
There was a problem hiding this comment.
rust-lang/stdarch#2261 should make this less crazy
All other architecture families currently use a single macro for both
32- and 64-bit versions. Do the same for MIPS.
Tracking issue: #111188
r? libs