search

riscv-non-isa / riscv-c-api-doc

Documentation of the RISC-V C API
https://jira.riscv.org/browse/RVG-4
Creative Commons Attribution 4.0 International
69 stars 39 forks source link

cbo Intrinsic #88

Open oferShinaar opened 1 week ago

oferShinaar commented 1 week ago

Is there a plan to add cbo Intrinsic to the doc

cmuellner commented 1 week ago

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

asb commented 1 week ago

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 commented 1 week ago

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

kito-cheng commented 1 week ago

Just a note to ensure we're all on the same page. The linked commit defines builtinriscv* 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 commented 1 week ago

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 commented 1 week ago

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!