[llvm] 232be51 - [NFC][llvm-exegesis] Fix comment text

Aiden Grossman via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 4 00:40:36 PST 2024


Author: Aiden Grossman
Date: 2024-01-04T00:40:30-08:00
New Revision: 232be5129b205b0eefee52b7ff930d597298bbfc

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

LOG: [NFC][llvm-exegesis] Fix comment text

I'm reasonably confident iff wasn't an abbreviation for if and only if
here and it was starting to bug me quite a bit.

Added: 
    

Modified: 
    llvm/tools/llvm-exegesis/llvm-exegesis.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/tools/llvm-exegesis/llvm-exegesis.cpp b/llvm/tools/llvm-exegesis/llvm-exegesis.cpp
index 1b35fde815f11f..ffbf94ce0fcb26 100644
--- a/llvm/tools/llvm-exegesis/llvm-exegesis.cpp
+++ b/llvm/tools/llvm-exegesis/llvm-exegesis.cpp
@@ -437,7 +437,7 @@ static void runBenchmarkConfigurations(
            ArrayRef<Benchmark>(AllResults).drop_front()) {
         llvm::append_range(Result.AssembledSnippet,
                            OtherResult.AssembledSnippet);
-        // Aggregate measurements, but only iff all measurements succeeded.
+        // Aggregate measurements, but only if all measurements succeeded.
         if (Result.Measurements.empty())
           continue;
         assert(OtherResult.Measurements.size() == Result.Measurements.size() &&


        


More information about the llvm-commits mailing list