-
Notifications
You must be signed in to change notification settings - Fork 100
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
[CIR][CIRGen][Builtin] Support __builtin_return_address #1046
base: main
Are you sure you want to change the base?
Conversation
1c5ea0b
to
e18d6a5
Compare
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.
Awesome, thanks! Just a minor nit and I'll merge it
|
||
def ReturnAddrOp : CIR_Op<"return_address"> { | ||
let arguments = (ins UInt32:$level); | ||
let summary = "call to `__builtin_return_address`"; |
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.
Summary should be a bit more generic, extracting from your full desc, one idea: The return address of the current function or of one of its callers
.
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.
Make sense, just updated the summary
✅ With the latest revision this PR passed the C/C++ code formatter. |
test case is from Traditional Clang CodeGen test file