Skip to content
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

cranelift-codegen: Rename abi::Caller to abi::CallSite #6414

Merged
merged 1 commit into from
May 19, 2023

Conversation

fitzgen
Copy link
Member

@fitzgen fitzgen commented May 18, 2023

Will be less confusing since each call site has both a caller and a callee.

@fitzgen fitzgen requested a review from a team as a code owner May 18, 2023 19:40
@fitzgen fitzgen requested review from cfallin and removed request for a team May 18, 2023 19:40
@fitzgen fitzgen changed the title cranelift-codegen: Rename abi::Callee to abi::CallSite cranelift-codegen: Rename abi::Caller to abi::CallSite May 18, 2023
@github-actions github-actions bot added cranelift Issues related to the Cranelift code generator cranelift:area:machinst Issues related to instruction selection and the new MachInst backend. cranelift:area:aarch64 Issues related to AArch64 backend. cranelift:area:x64 Issues related to x64 codegen isle Related to the ISLE domain-specific language labels May 18, 2023
@github-actions
Copy link

Subscribe to Label Action

cc @cfallin, @fitzgen

This issue or pull request has been labeled: "cranelift", "cranelift:area:aarch64", "cranelift:area:machinst", "cranelift:area:x64", "isle"

Thus the following users have been cc'd because of the following labels:

  • cfallin: isle
  • fitzgen: isle

To subscribe or unsubscribe from this label, edit the .github/subscribe-to-label.json configuration file.

Learn more.

@jameysharp
Copy link
Contributor

Registering a drive-by vote in favor of this idea, but I'd still like @cfallin's review.

Will be less confusing since each call site has both a caller and a callee.
Copy link
Member

@cfallin cfallin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems fine to me; @fitzgen and I talked it over and tried to brainstorm better names a bit yesterday and this is I think the clearest option.

Just to document a little history here: Caller (née ABICaller) was previously ABICall and Callee (née ABICallee) was previously ABIBody; the last rename happened in #2142 after some review comments; specifically here was where caller/callee were suggested, because "call" was confusing. Perhaps "callsite" is a bit better / more explicit though.

@fitzgen fitzgen added this pull request to the merge queue May 18, 2023
@alexcrichton
Copy link
Member

Oh merge queue is configured for merge instead of squash right now so I'm going to bump this out while pat lands #6391

@alexcrichton alexcrichton removed this pull request from the merge queue due to a manual request May 18, 2023
@alexcrichton alexcrichton added this pull request to the merge queue May 19, 2023
Merged via the queue into bytecodealliance:main with commit 28931a4 May 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cranelift:area:aarch64 Issues related to AArch64 backend. cranelift:area:machinst Issues related to instruction selection and the new MachInst backend. cranelift:area:x64 Issues related to x64 codegen cranelift Issues related to the Cranelift code generator isle Related to the ISLE domain-specific language
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants