[llvm] f534992 - Fix: Reordering parameters in getFile and getFileOrSTDIN

Abhina Sreeskantharajan via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 25 08:56:04 PDT 2021


Author: Abhina Sreeskantharajan
Date: 2021-03-25T11:55:57-04:00
New Revision: f5349922c06fec0feec53f9eab90aaaa5fe6ed17

URL: https://github.com/llvm/llvm-project/commit/f5349922c06fec0feec53f9eab90aaaa5fe6ed17
DIFF: https://github.com/llvm/llvm-project/commit/f5349922c06fec0feec53f9eab90aaaa5fe6ed17.diff

LOG: Fix: Reordering parameters in getFile and getFileOrSTDIN

There was a new getFileOrSTDIN call added recently which was not included in my patch. https://reviews.llvm.org/D99110
I reordered the args to match the new order.

Reviewed By: tunz

Differential Revision: https://reviews.llvm.org/D99349

Added: 
    

Modified: 
    llvm/lib/ProfileData/Coverage/CoverageMapping.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/ProfileData/Coverage/CoverageMapping.cpp b/llvm/lib/ProfileData/Coverage/CoverageMapping.cpp
index 1958e310f4fc..f81394612190 100644
--- a/llvm/lib/ProfileData/Coverage/CoverageMapping.cpp
+++ b/llvm/lib/ProfileData/Coverage/CoverageMapping.cpp
@@ -329,7 +329,7 @@ CoverageMapping::load(ArrayRef<StringRef> ObjectFilenames,
 
   for (const auto &File : llvm::enumerate(ObjectFilenames)) {
     auto CovMappingBufOrErr = MemoryBuffer::getFileOrSTDIN(
-        File.value(), /*FileSize=*/-1, /*RequiresNullTerminator=*/false);
+        File.value(), /*IsText=*/false, /*RequiresNullTerminator=*/false);
     if (std::error_code EC = CovMappingBufOrErr.getError())
       return errorCodeToError(EC);
     StringRef Arch = Arches.empty() ? StringRef() : Arches[File.index()];


        


More information about the llvm-commits mailing list