Fix a leak in `ASTUnit::LoadFromCommandLine()` wehn compiler invocation fails

[bstaletic]

Summary

1. Here we allocate buffers to hold the contents of unsaved files.
2. The RemappedFiles vector is then passed to ASTUnit::LoadFromCommandLine() as an ArrayRef, here
3. In ASTUnit::LoadFromCommandLine() we try to make a compiler invocation and exit early if that fails.
4. If we did exit early from that function, both Unit and UnitErr are nullptr, making the function exit without ever cleaning up the memory allocations from step 1.

Notes:

1. The function allocating all those RemappedFile objects is not actually the same as the one deallocating, in the happy case.

1.1. I did not figure out where exactly does it get deleted, because the above seems to be enough data for now.

2. My attempted solution is to iterate over remapped files and deallocate in this block, but it seems odd to deallocate through ArrayRef.

Fixes https://bugs.llvm.org/show_bug.cgi?id=47832

[bstaletic]

Fixed the code formatting error.

clang/lib/Frontend/ASTUnit.cpp

ASTUnit *ASTUnit::LoadFromCommandLine(
   std::shared_ptr<CompilerInvocation> CI;
 
   {
     CaptureDroppedDiagnostics Capture(CaptureDiagnostics, *Diags,
                                       &StoredDiagnostics, nullptr);
 
     CI = createInvocationFromCommandLine(
         llvm::makeArrayRef(ArgBegin, ArgEnd), Diags, VFS);
     if (!CI)
+    {
+      for (const auto &RF : RemappedFiles)
+        delete RF.second;
       return nullptr;
     }
+  }
 
   // Override any files that need remapping
   for (const auto &RemappedFile : RemappedFiles) {
     CI->getPreprocessorOpts().addRemappedFile(RemappedFile.first,
                                               RemappedFile.second);
   }
   PreprocessorOptions &PPOpts = CI->getPreprocessorOpts();
   PPOpts.RemappedFilesKeepOriginalName = RemappedFilesKeepOriginalName;