github-actions[bot] commented on Issue #2149:
Subscribe to Label Action
cc @bnjbvr
<details>
This issue or pull request has been labeled: "cranelift", "cranelift:area:aarch64", "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 <code>.github/subscribe-to-label.json</code> configuration file.
Learn more.
</details>
julian-seward1 commented on Issue #2149:
As a comment to reviewers:
enum AtomicRMWOp
inaarch64/inst/args.rs
has been cloned exactly intox64/inst/args.rs
. I'm not happy about the duplication and would prefer to have only one copy. But where? -- do we have a place where we can put machine-insn artefacts, that are shared across multiple architectures?
bjorn3 commented on Issue #2149:
Somewhere in
cranelift/codegen/src/machinst
probably.
julian-seward1 commented on Issue #2149:
All review comments have been addressed.
bnjbvr commented on Issue #2149:
One extra comment about the commit message: can you use a shorter one liner for the first line, like
machinst x64: implement atomic opcodes
, and keep the rest of the commit message in a paragraph below the first line? It's a bit unusual to have a whole paragraph as the commit message, and it doesn't display nicely in misc UIs (github/git).
Last updated: Dec 23 2024 at 13:07 UTC