Skip to content

x64: Begin migrating branch instructions to ISLE#4587

Merged
elliottt merged 6 commits intobytecodealliance:mainfrom
elliottt:trevor/x64-isle-br
Aug 3, 2022
Merged

x64: Begin migrating branch instructions to ISLE#4587
elliottt merged 6 commits intobytecodealliance:mainfrom
elliottt:trevor/x64-isle-br

Conversation

@elliottt
Copy link
Copy Markdown
Member

@elliottt elliottt commented Aug 3, 2022

Begin migrating branch instructions for the x64 backend to ISLE.

@github-actions github-actions bot added cranelift Issues related to the Cranelift code generator cranelift:area:x64 Issues related to x64 codegen isle Related to the ISLE domain-specific language labels Aug 3, 2022
@github-actions
Copy link
Copy Markdown

github-actions bot commented Aug 3, 2022

Subscribe to Label Action

cc @cfallin, @fitzgen

Details This issue or pull request has been labeled: "cranelift", "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.

Copy link
Copy Markdown
Member

@fitzgen fitzgen left a comment

Choose a reason for hiding this comment

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

Looks great!

Comment on lines +520 to +522
(decl single_target (MachLabel) MachLabelSlice)
(extern extractor single_target single_target)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Could you add a little doc comment for this? Something like

;; Match a label slice of length one, extracting the only label in the slice.

Comment on lines +523 to +526
(decl two_targets (MachLabel MachLabel) MachLabelSlice)
(extern extractor two_targets two_targets)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Similar doc comment here please.

@elliottt elliottt marked this pull request as ready for review August 3, 2022 19:01
@elliottt elliottt force-pushed the trevor/x64-isle-br branch from 8a3679f to c05f820 Compare August 3, 2022 19:02
@elliottt elliottt force-pushed the trevor/x64-isle-br branch from c05f820 to 97c84b8 Compare August 3, 2022 19:08
@elliottt elliottt enabled auto-merge (squash) August 3, 2022 20:15
@elliottt elliottt merged commit 301be74 into bytecodealliance:main Aug 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

2 participants