review comments fix

Author: midhuncodes7

llvm/lib/DebugInfo/Symbolize/Symbolize.cpp

@@ -579,7 +579,6 @@ LLVMSymbolizer::loadOrGetBinary(const std::string &Path) {
 
   Expected<OwningBinary<Binary>> BinOrErr = createBinary(Path);
   if (!BinOrErr) {
-    BinaryForPath.erase(Pair.first);
     return BinOrErr.takeError();
   }
 

llvm/test/DebugInfo/symbolize-big-archive-elf.test

@@ -25,7 +25,3 @@
 // CHECK-NOTARCHIVE: error: '{{.*}}' is not a valid archive
 // RUN: not llvm-symbolizer --obj="%t.a()" 0x1000 2>&1 | FileCheck %s --check-prefix=CHECK-EMPTY
 // CHECK-EMPTY: {{.*}}no matching member{{.*}}
-// RUN: not llvm-symbolizer --obj="%t.a(%t.o" 0x1000 2>&1 | FileCheck %s --check-prefix=CHECK-INCORRECT-FORMAT-1
-// CHECK-INCORRECT-FORMAT-1: {{.*}}No such file or directory{{.*}}
-// RUN: not llvm-symbolizer --obj="%t.a%t.o)" 0x1000 2>&1 | FileCheck %s --check-prefix=CHECK-INCORRECT-FORMAT-2
-// CHECK-INCORRECT-FORMAT-2: {{.*}}Not a directory{{.*}}

llvm/test/DebugInfo/symbolize-big-archive-xcoff.test

@@ -33,8 +33,3 @@
 // CHECK-NOTARCHIVE: error: '{{.*}}' is not a valid archive
 // RUN: not llvm-symbolizer --obj="%t.a()" 0x1000 2>&1 | FileCheck %s --check-prefix=CHECK-EMPTY
 // CHECK-EMPTY: {{.*}}no matching member{{.*}}
-// RUN: not llvm-symbolizer --obj="%t.a(%t.o" 0x1000 2>&1 | FileCheck %s --check-prefix=CHECK-INCORRECT-FORMAT-1
-// CHECK-INCORRECT-FORMAT-1: {{.*}}No such file or directory{{.*}}
-// RUN: not llvm-symbolizer --obj="%t.a%t.o)" 0x1000 2>&1 | FileCheck %s --check-prefix=CHECK-INCORRECT-FORMAT-2
-// CHECK-INCORRECT-FORMAT-2: {{.*}}Not a directory{{.*}}
-