[llvm] [llvm-ifs] Handle more e_machine values for --target (PR #128559)

Saleem Abdulrasool via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 24 12:39:29 PST 2025


================
@@ -200,6 +200,71 @@ uint16_t ELF::convertArchNameToEMachine(StringRef Arch) {
       .Default(EM_NONE);
 }
 
+uint16_t ELF::convertTripleArchTypeToEMachine(Triple::ArchType ArchType) {
+  switch (ArchType) {
+  case Triple::UnknownArch:
+  default:
+    return EM_NONE;
----------------
compnerd wrote:

Can we leave `default` out? I think that we want a covered switch?

https://github.com/llvm/llvm-project/pull/128559


More information about the llvm-commits mailing list