Skip to content

Conversation

@antoniofrighetto
Copy link
Contributor

We have explained how musttail can be guaranteed when the calling convention is not swifttailcc or tailcc, ensure what needs to adhere when it is the opposite case (reflect how it is already handled in Verifier).

@llvmbot
Copy link
Member

llvmbot commented Oct 5, 2024

@llvm/pr-subscribers-llvm-ir

Author: Antonio Frighetto (antoniofrighetto)

Changes

We have explained how musttail can be guaranteed when the calling convention is not swifttailcc or tailcc, ensure what needs to adhere when it is the opposite case (reflect how it is already handled in Verifier).


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

1 Files Affected:

  • (modified) llvm/docs/LangRef.rst (+1-1)
diff --git a/llvm/docs/LangRef.rst b/llvm/docs/LangRef.rst
index 6fa35486669d69..1dd4279efcfb68 100644
--- a/llvm/docs/LangRef.rst
+++ b/llvm/docs/LangRef.rst
@@ -12747,7 +12747,7 @@ This instruction requires several arguments:
    - The caller and callee prototypes must match. Pointer types of parameters
      or return types may differ in pointee type, but not in address space.
 
-  On the other hand, if the calling convention is `swifttailcc` or `swiftcc`:
+  On the other hand, if the calling convention is `swifttailcc` or `tailcc`:
 
    - Only these ABI-impacting attributes attributes are allowed: sret, byval,
      swiftself, and swiftasync.

@antoniofrighetto
Copy link
Contributor Author

Kind ping @TNorthover as original author.

@antoniofrighetto
Copy link
Contributor Author

Kind ping.

@antoniofrighetto antoniofrighetto force-pushed the feature/langref-tailcc-attrs branch from a5323a5 to 11b9bd4 Compare November 18, 2024 09:57
We have explained how musttail can be guaranteed when the calling
convention is not `swifttailcc` or `tailcc`, ensure what needs to
adhere when it is the opposite case.
@antoniofrighetto antoniofrighetto force-pushed the feature/langref-tailcc-attrs branch from 11b9bd4 to 9fc4654 Compare November 18, 2024 10:02
@antoniofrighetto antoniofrighetto merged commit 9fc4654 into llvm:main Nov 18, 2024
6 of 9 checks passed
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