[dsymutil] Add DW_TAG_common_block to dieNeedsChildrenToBeMeaningful

Ensure we walk the children of common blocks when deciding what DIEs to
keep. Otherwise we might incorrectly discard them leading to missing
variables in the linked debug info.

This also sorts the list of DW_TAGs alphabetically.

(cherry picked from commit 96bbf47)

Author: JDevlieghere

llvm/tools/dsymutil/DwarfLinker.cpp

@@ -359,11 +359,12 @@ static bool dieNeedsChildrenToBeMeaningful(uint32_t Tag) {
   switch (Tag) {
   default:
     return false;
-  case dwarf::DW_TAG_subprogram:
+  case dwarf::DW_TAG_class_type:
+  case dwarf::DW_TAG_common_block:
   case dwarf::DW_TAG_lexical_block:
-  case dwarf::DW_TAG_subroutine_type:
   case dwarf::DW_TAG_structure_type:
-  case dwarf::DW_TAG_class_type:
+  case dwarf::DW_TAG_subprogram:
+  case dwarf::DW_TAG_subroutine_type:
   case dwarf::DW_TAG_union_type:
     return true;
   }