[llvm-readobj] Change "SHT_MIPS_DWARF" to "MIPS_DWARF"

[MaskRay]

Summary

This is to be consistent with the display of other MIPS section types.
This string is also used by binutils-gdb/binutils/readelf.c:get_mips_section_type_name

Since we are here, reorder the two enum constatns because SHT_MIPS_DWARF < SHT_MIPS_ABIFLAGS.

[jhenderson]

I'd normally ask for a test, but as D58457 has yet to land, I'll fix that up!

tools/llvm-readobj/ELFDumper.cpp

2863–2864 ↗

(On Diff #187747)

Is there a good reason to reorder this? For diff-ability, it would be better if the ordering is maintained.

[MaskRay]

tools/llvm-readobj/ELFDumper.cpp

2863–2864 ↗

(On Diff #187747)

Because SHT_MIPS_DWARF < SHT_MIPS_ABIFLAGS and it is the order used in several other places. I can update the description to mention this.

[jhenderson]

Okay, LGTM.

[MaskRay]

.

llvm/trunk/tools/llvm-readobj/ELFDumper.cpp

case EM_X86_64:
     break;
   case EM_MIPS:
   case EM_MIPS_RS3_LE:
     switch (Type) {
     case SHT_MIPS_REGINFO:
       return "MIPS_REGINFO";
     case SHT_MIPS_OPTIONS:
       return "MIPS_OPTIONS";
+    case SHT_MIPS_DWARF:
+      return "MIPS_DWARF";
     case SHT_MIPS_ABIFLAGS:
       return "MIPS_ABIFLAGS";
-    case SHT_MIPS_DWARF:
-      return "SHT_MIPS_DWARF";
     }
     break;
   }
   switch (Type) {
   case SHT_NULL:
     return "NULL";
   case SHT_PROGBITS:
     return "PROGBITS";