Skip to content

Add pbc32 and pbc64 architectures for Pulley#109

Merged
fitzgen merged 2 commits intobytecodealliance:mainfrom
fitzgen:pulley
Jul 30, 2024
Merged

Add pbc32 and pbc64 architectures for Pulley#109
fitzgen merged 2 commits intobytecodealliance:mainfrom
fitzgen:pulley

Conversation

@fitzgen
Copy link
Member

@fitzgen fitzgen commented Jul 29, 2024

"pbc" = Pulley Bytecode. The 32 and 64 refer to the pointer size.

"pbc" = Pulley Bytecode. The 32 and 64 refer to the pointer size.
@bjorn3
Copy link
Contributor

bjorn3 commented Jul 29, 2024

Maybe call them pulley32 and pulley64 instead? These are not targets supported by rustc, so favoring the name being unambiguous over it being short is the right call IMHO.

Copy link
Member

@alexcrichton alexcrichton left a comment

Choose a reason for hiding this comment

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

I think @bjorn3's suggestion is reasonable here as well and might help disambiguate from other targets too.

@fitzgen fitzgen merged commit da9f879 into bytecodealliance:main Jul 30, 2024
@fitzgen fitzgen deleted the pulley branch July 30, 2024 22:34
@fitzgen fitzgen mentioned this pull request Jul 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants