LLVM patch: Fix Triple::isTargetEHABICompatible() for NetBSD

alexrp · alexrp · commit 25a76f8265b6 · 2025-06-10T17:16:45.000+02:00
This can hopefully be dropped with LLVM 21.
diff --git a/README.md b/README.md
@@ -21,6 +21,7 @@ For other versions, check the git tags of this repository.
  * LLVM: Support .lib extension for static zstd.
  * LLVM: Portable handling of .def linker flag
  * LLVM: Don't pass -static when building executables.
+ * LLVM: Fix `Triple::isTargetEHABICompatible()` for NetBSD
  * Clang: Ignore the examples directory
  * Clang: Disable building of libclang-cpp.so.
  * LLD: Added additional include directory to Zig's libunwind.
diff --git a/llvm/include/llvm/TargetParser/Triple.h b/llvm/include/llvm/TargetParser/Triple.h
@@ -912,7 +912,8 @@ class Triple {
             getEnvironment() == Triple::GNUEABIHFT64 ||
             getEnvironment() == Triple::OpenHOS ||
             getEnvironment() == Triple::MuslEABIHF || isAndroid()) &&
-           isOSBinFormatELF();
+           isOSBinFormatELF() &&
+           !isOSNetBSD();
   }
 
   /// Tests whether the target is T32.

Original file line number	Diff line number	Diff line change
`@@ -912,7 +912,8 @@ class Triple {`
`912`	`912`	`getEnvironment() == Triple::GNUEABIHFT64 \|\|`
`913`	`913`	`getEnvironment() == Triple::OpenHOS \|\|`
`914`	`914`	`getEnvironment() == Triple::MuslEABIHF \|\| isAndroid()) &&`
`915`		`- isOSBinFormatELF();`
	`915`	`+ isOSBinFormatELF() &&`
	`916`	`+ !isOSNetBSD();`
`916`	`917`	`}`
`917`	`918`
`918`	`919`	`/// Tests whether the target is T32.`