[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