removed check for DWARF5, it will pre-load DWARF4 also.

Author: ayermolo

llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp

@@ -1930,22 +1930,18 @@ void DWARFVerifier::verifyNameIndexCompleteness(
 static void extractCUsTus(DWARFContext &DCtx) {
   // Abbrev DeclSet is shared beween the units.
   for (auto &CUTU : DCtx.normal_units()) {
-    if (CUTU->getVersion() < 5)
-      continue;
     CUTU->getUnitDIE();
     CUTU->getBaseAddress();
   }
   parallelForEach(DCtx.normal_units(), [&](const auto &CUTU) {
-    if (CUTU->getVersion() < 5)
-      return;
     if (Error E = CUTU->tryExtractDIEsIfNeeded(false))
       DCtx.getRecoverableErrorHandler()(std::move(E));
   });
 
   // Invoking getNonSkeletonUnitDIE() sets up all the base pointers for DWO
   // Units. This is needed for getBaseAddress().
   for (const auto &CU : DCtx.compile_units()) {
-    if (!(CU->getVersion() >= 5 && CU->getDWOId()))
+    if (!CU->getDWOId())
       continue;
     DWARFContext &NonSkeletonContext =
         CU->getNonSkeletonUnitDIE().getDwarfUnit()->getContext();

llvm/test/tools/llvm-dwarfdump/X86/verify_no_linkage_name.s

@@ -1,8 +1,6 @@
 # This test generates a DW_TAG_structure_type with a linkage name. This linkage
 # name will not be part of the accelerator table and the verifier should not
 # complain about this.
-# Technically speaking this test is invalid because .debug_names is a DWARF 5 feature.
-# In this test the accelerator table is referencing DWARF4.
 #
 # DW_TAG_structure_type
 #   DW_AT_name    ("C")
@@ -11,7 +9,7 @@
 # RUN: llvm-mc %s -filetype obj -triple x86_64-unknown-linux-gnu -o %t.o
 # RUN: llvm-dwarfdump -debug-info %t.o | FileCheck %s
 # RUN: llvm-dwarfdump -debug-names %t.o | FileCheck %s --check-prefix ACCEL
-# RUN: llvm-dwarfdump -verify --verify-num-threads 1 -debug-names %t.o
+# RUN: llvm-dwarfdump -verify -debug-names %t.o
 
 # CHECK: DW_AT_name    ("Bool")
 # CHECK-NEXT: DW_AT_linkage_name    ("$SSbD")