[llvm] [SimplifyCFG] Better aligned a comment. (PR #109307)

Simone Campanoni via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 30 06:52:48 PDT 2024


https://github.com/scampanoni updated https://github.com/llvm/llvm-project/pull/109307

>From 3a2e0feaf2878993080ea2944d5d63de364a8950 Mon Sep 17 00:00:00 2001
From: Simone Campanoni <campanoni at google.com>
Date: Thu, 19 Sep 2024 16:14:12 +0000
Subject: [PATCH 1/2] [SimplifyCFG] Better aligned a comment.

---
 llvm/lib/Transforms/Utils/SimplifyCFG.cpp | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/llvm/lib/Transforms/Utils/SimplifyCFG.cpp b/llvm/lib/Transforms/Utils/SimplifyCFG.cpp
index c63618d9dd1297..2046ac4cc3ef52 100644
--- a/llvm/lib/Transforms/Utils/SimplifyCFG.cpp
+++ b/llvm/lib/Transforms/Utils/SimplifyCFG.cpp
@@ -405,9 +405,9 @@ static InstructionCost computeSpeculationCost(const User *I,
 }
 
 /// If we have a merge point of an "if condition" as accepted above,
-/// return true if the specified value dominates the block.  We
-/// don't handle the true generality of domination here, just a special case
-/// which works well enough for us.
+/// return true if the specified value dominates the block.  We don't handle
+/// the true generality of domination here, just a special case which works 
+/// well enough for us.
 ///
 /// If AggressiveInsts is non-null, and if V does not dominate BB, we check to
 /// see if V (which must be an instruction) and its recursive operands

>From 4ee5b91966882806bcf17bcfe208a06f65ffa349 Mon Sep 17 00:00:00 2001
From: Simone Campanoni <campanoni at google.com>
Date: Mon, 30 Sep 2024 13:50:12 +0000
Subject: [PATCH 2/2] [SimplifyCFG] Run the git-clang-format

---
 llvm/lib/Transforms/Utils/SimplifyCFG.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/llvm/lib/Transforms/Utils/SimplifyCFG.cpp b/llvm/lib/Transforms/Utils/SimplifyCFG.cpp
index 2046ac4cc3ef52..08318b6788c289 100644
--- a/llvm/lib/Transforms/Utils/SimplifyCFG.cpp
+++ b/llvm/lib/Transforms/Utils/SimplifyCFG.cpp
@@ -406,7 +406,7 @@ static InstructionCost computeSpeculationCost(const User *I,
 
 /// If we have a merge point of an "if condition" as accepted above,
 /// return true if the specified value dominates the block.  We don't handle
-/// the true generality of domination here, just a special case which works 
+/// the true generality of domination here, just a special case which works
 /// well enough for us.
 ///
 /// If AggressiveInsts is non-null, and if V does not dominate BB, we check to



More information about the llvm-commits mailing list