Skip to content

machinst x64: a few small refactorings/renamings#2189

Merged
cfallin merged 7 commits intobytecodealliance:mainfrom
bnjbvr:x64-refactor-sub
Sep 9, 2020
Merged

machinst x64: a few small refactorings/renamings#2189
cfallin merged 7 commits intobytecodealliance:mainfrom
bnjbvr:x64-refactor-sub

Conversation

@bnjbvr
Copy link
Member

@bnjbvr bnjbvr commented Sep 9, 2020

Small refactorings I intended to do; more to come. No functional changes here.

@bnjbvr bnjbvr requested a review from cfallin September 9, 2020 16:05
@bjorn3
Copy link
Contributor

bjorn3 commented Sep 9, 2020

and this would require abstractions brought up in another pending PR#2142.

#2142 has already been merged. Did you mean another PR?

@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 labels Sep 9, 2020
@github-actions
Copy link

github-actions bot commented Sep 9, 2020

Subscribe to Label Action

cc @bnjbvr

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

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

  • bnjbvr: cranelift

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

Learn more.

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.

Good cleanups; thanks!

@cfallin cfallin merged commit bd3ba0a into bytecodealliance:main Sep 9, 2020
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants