Run GCF

Author: Jlalond

llvm/include/llvm/Object/Minidump.h

@@ -183,7 +183,6 @@ class MinidumpFile : public Binary {
             object_error::unexpected_eof);
       }
 
-
       ArrayRef<uint8_t> Content = Storage.take_front(Descriptor.DataSize);
       Current = std::make_pair(Descriptor, Content);
 
@@ -200,9 +199,9 @@ class MinidumpFile : public Binary {
     // if the first descriptor is readable.
     Memory64Iterator(ArrayRef<uint8_t> Storage,
                      ArrayRef<minidump::MemoryDescriptor_64> Descriptors)
-        : Storage(Storage), Descriptors(Descriptors),
-          IsEnd(false) {
-      assert(Descriptors.size() > 0 && Storage.size() >= Descriptors.front().DataSize);
+        : Storage(Storage), Descriptors(Descriptors), IsEnd(false) {
+      assert(Descriptors.size() > 0 &&
+             Storage.size() >= Descriptors.front().DataSize);
       minidump::MemoryDescriptor_64 Descriptor = Descriptors.front();
       ArrayRef<uint8_t> Content = Storage.take_front(Descriptor.DataSize);
       Current = std::make_pair(Descriptor, Content);
@@ -226,8 +225,7 @@ class MinidumpFile : public Binary {
   /// content from the Memory64List stream. An error is returned if the file
   /// does not contain a Memory64List stream, or if the descriptor data is
   /// unreadable.
-  iterator_range<FallibleMemory64Iterator>
-  getMemory64List(Error &Err) const;
+  iterator_range<FallibleMemory64Iterator> getMemory64List(Error &Err) const;
 
   /// Returns the list of descriptors embedded in the MemoryInfoList stream. The
   /// descriptors provide properties (e.g. permissions) of interesting regions

llvm/lib/Object/Minidump.cpp

@@ -156,9 +156,13 @@ MinidumpFile::create(MemoryBufferRef Source) {
       new MinidumpFile(Source, Hdr, *ExpectedStreams, std::move(StreamMap)));
 }
 
-static iterator_range<MinidumpFile::FallibleMemory64Iterator> makeEmptyRange(Error &Err) {
-    return make_range(llvm::object::MinidumpFile::FallibleMemory64Iterator::itr(llvm::object::MinidumpFile::Memory64Iterator::end(), Err),
-                      llvm::object::MinidumpFile::FallibleMemory64Iterator::end(llvm::object::MinidumpFile::Memory64Iterator::end()));
+static iterator_range<MinidumpFile::FallibleMemory64Iterator>
+makeEmptyRange(Error &Err) {
+  return make_range(
+      llvm::object::MinidumpFile::FallibleMemory64Iterator::itr(
+          llvm::object::MinidumpFile::Memory64Iterator::end(), Err),
+      llvm::object::MinidumpFile::FallibleMemory64Iterator::end(
+          llvm::object::MinidumpFile::Memory64Iterator::end()));
 }
 
 iterator_range<MinidumpFile::FallibleMemory64Iterator>
@@ -192,9 +196,9 @@ MinidumpFile::getMemory64List(Error &Err) const {
     return makeEmptyRange(Err);
   }
 
-  return make_range(
-      FallibleMemory64Iterator::itr(
-          Memory64Iterator::begin(getData().slice(ListHeader->BaseRVA), *Descriptors),
-          Err),
-      FallibleMemory64Iterator::end(Memory64Iterator::end()));
+  return make_range(FallibleMemory64Iterator::itr(
+                        Memory64Iterator::begin(
+                            getData().slice(ListHeader->BaseRVA), *Descriptors),
+                        Err),
+                    FallibleMemory64Iterator::end(Memory64Iterator::end()));
 }

llvm/lib/ObjectYAML/MinidumpYAML.cpp

@@ -535,7 +535,7 @@ Stream::create(const Directory &StreamDesc, const object::MinidumpFile &File) {
       return Err;
     std::vector<Memory64ListStream::entry_type> Ranges;
     for (const auto &Pair : Memory64List) {
-        Ranges.push_back({Pair.first, Pair.second});
+      Ranges.push_back({Pair.first, Pair.second});
     }
 
     // If we don't have an error, or if any of the reads succeed, return ranges

llvm/unittests/ObjectYAML/MinidumpYAMLTest.cpp

@@ -367,8 +367,6 @@ TEST(MinidumpYAML, MemoryRegion_64bit) {
       *ExpectedMemoryList;
   auto Iterator = MemoryList.begin();
 
-
-
   auto DescOnePair = *Iterator;
   const minidump::MemoryDescriptor_64 &DescOne = DescOnePair.first;
   ASSERT_THAT(DescOne.StartOfMemoryRange, 0x7FFFFFCF0818283u);