-
Notifications
You must be signed in to change notification settings - Fork 14.3k
[LangRef] Document the fourth operand for vp.cttz/ctlz. NFC #97387
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
@llvm/pr-subscribers-llvm-ir Author: Craig Topper (topperc) ChangesFull diff: https://github.com/llvm/llvm-project/pull/97387.diff 1 Files Affected:
diff --git a/llvm/docs/LangRef.rst b/llvm/docs/LangRef.rst
index e2c47204e628f..ed988e48daa09 100644
--- a/llvm/docs/LangRef.rst
+++ b/llvm/docs/LangRef.rst
@@ -24503,7 +24503,9 @@ Arguments:
The first operand and the result have the same vector of integer type. The
second operand is the vector mask and has the same number of elements as the
result vector type. The third operand is the explicit vector length of the
-operation.
+operation. The fourth operand is a constant flag that indicates whether the
+intrinsic returns a valid result if the first operand is zero. If the first
+operand is zero and the fourth operand is true, the result is poison.
Semantics:
""""""""""
@@ -24550,7 +24552,9 @@ Arguments:
The first operand and the result have the same vector of integer type. The
second operand is the vector mask and has the same number of elements as the
result vector type. The third operand is the explicit vector length of the
-operation.
+operation. The fourth operand is a constant flag that indicates whether the
+intrinsic returns a valid result if the first operand is zero. If the first
+operand is zero and the fourth operand is true, the result is poison.
Semantics:
""""""""""
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - with one minor peeve - the subsection is called "Arguments:" but then we refer to them as operands :\
Yeah I noticed that too. Its all over the VP intrinsics. Maybe I'll do a bulk rename. |
…s. NFC Fixes inconsistencies noted in llvm#97386 and llvm#97387. There are still more intrinsics that have the same issue. I might submit more patches for those.
…sics. NFC (llvm#97437) Fixes inconsistencies noted in llvm#97386 and llvm#97387. There are still more intrinsics that have the same issue. I might submit more patches for those.
…sics. NFC (llvm#97437) Fixes inconsistencies noted in llvm#97386 and llvm#97387. There are still more intrinsics that have the same issue. I might submit more patches for those.
No description provided.