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

cbo Intrinsic #88

Open
oferShinaar opened this issue Sep 24, 2024 · 7 comments
Open

cbo Intrinsic #88

oferShinaar opened this issue Sep 24, 2024 · 7 comments

Comments

@oferShinaar
Copy link

Is there a plan to add cbo Intrinsic
to the doc

@cmuellner
Copy link
Collaborator

I think nobody is working on this.
So, a PR would be appreciated!

@asb
Copy link
Contributor

asb commented Sep 24, 2024

Just a note to ensure we're all on the same page. The linked commit defines __builtin_riscv_* functions which so far have been considered out of the scope of this document (though I think we typically try to match what gcc does in LLVM). So if it's desirable, perhaps the path forward is to define a __riscv_* intrinsic that could be standardised.

(Christoph / Kito / ... - please correct me if wrong!)

@cmuellner
Copy link
Collaborator

Yes, I was expecting to get a PR that would follow the naming convention in the document. Thanks for stating this clearly!

@kito-cheng
Copy link
Collaborator

Just a note to ensure we're all on the same page. The linked commit defines _builtin_riscv* functions which so far have been considered out of the scope of this document (though I think we typically try to match what gcc does in LLVM). So if it's desirable, perhaps the path forward is to define a _riscv* intrinsic that could be standardised.

Thanks my expectation as well :)

@amitch1999
Copy link

Just a note to ensure we're all on the same page. The linked commit defines __builtin_riscv_* functions which so far have been considered out of the scope of this document (though I think we typically try to match what gcc does in LLVM). So if it's desirable, perhaps the path forward is to define a __riscv_* intrinsic that could be standardised.

(Christoph / Kito / ... - please correct me if wrong!)

Might you know if those intrinsics are a wip in LLVM? There seems to be only assembly support for Zicbom, and I couldn't find a PR or issue regarding implementing these.

@asb
Copy link
Contributor

asb commented Sep 24, 2024

Might you know if those intrinsics are a wip in LLVM? There seems to be only assembly support for Zicbom, and I couldn't find a PR or issue regarding implementing these.

Nobody's posted a PR to my knowledge, and I'm not aware of anyone working on them so far. Patches welcome!

@yulong18
Copy link

I try to send a patchset to upstream.
My work based on doc.

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

No branches or pull requests

6 participants