Skip to content

[LangRef] fix non-existant icmp gte -> icmp sge#149420

Merged
fmayer merged 1 commit intomainfrom
users/fmayer/spr/langref-fix-non-existant-icmp-gte-icmp-sge
Jul 17, 2025
Merged

[LangRef] fix non-existant icmp gte -> icmp sge#149420
fmayer merged 1 commit intomainfrom
users/fmayer/spr/langref-fix-non-existant-icmp-gte-icmp-sge

Conversation

@fmayer
Copy link
Copy Markdown
Contributor

@fmayer fmayer commented Jul 17, 2025

No description provided.

Created using spr 1.3.4
@llvmbot
Copy link
Copy Markdown
Member

llvmbot commented Jul 17, 2025

@llvm/pr-subscribers-llvm-ir

Author: Florian Mayer (fmayer)

Changes

Full diff: https://github.com/llvm/llvm-project/pull/149420.diff

1 Files Affected:

  • (modified) llvm/docs/LangRef.rst (+1-1)
diff --git a/llvm/docs/LangRef.rst b/llvm/docs/LangRef.rst
index 2759e18301d58..371f356c80b0a 100644
--- a/llvm/docs/LangRef.rst
+++ b/llvm/docs/LangRef.rst
@@ -4867,7 +4867,7 @@ to be eliminated. This is because '``poison``' is stronger than '``undef``'.
 
       %D = undef
       %E = icmp slt %D, 4
-      %F = icmp gte %D, 4
+      %F = icmp sge %D, 4
 
     Safe:
       %A = undef

@fmayer fmayer requested a review from arsenm July 17, 2025 22:54
@fmayer fmayer merged commit 547a49f into main Jul 17, 2025
12 checks passed
@fmayer fmayer deleted the users/fmayer/spr/langref-fix-non-existant-icmp-gte-icmp-sge branch July 17, 2025 23:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants