This is an archive of the discontinued LLVM Phabricator instance.

[libcxx] Add OHOS support
AbandonedPublic

Authored by ldionne on Mar 24 2023, 4:42 AM.

Details

Reviewers
phosek
thesamesam
Mordante
kpdev42
Group Reviewers
Restricted Project
Summary

Add support for OpenHarmony OS (OHOS)

OpenHarmony is an open-source project incubated and operated by the OpenAtom Foundation. It targets IoT, and already used on production devices.

General OpenHarmony OS discussion on discourse thread: "Add support for OpenHarmony OS"

LLVM and clang parts of OHOS support: D138202, D145227

~~

Huawei RRI, OS Lab

Diff Detail

Event Timeline

kpdev42 created this revision.Mar 24 2023, 4:42 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 24 2023, 4:42 AM
Herald added a subscriber: krytarowski. · View Herald Transcript
kpdev42 requested review of this revision.Mar 24 2023, 4:42 AM
Herald added a reviewer: Restricted Project. · View Herald TranscriptMar 24 2023, 4:42 AM

As usual: Do you plan to get official support for OpenHarmony in libc++? If that's the case, please take a look at https://libcxx.llvm.org/AddingNewCIJobs.html#addingnewcijobs.

Can you update the commit message to explain what OHOS is and provide a link to its homepage. That would be nice to have as a future reference.

The atomic changes look fine.

kpdev42 edited the summary of this revision. (Show Details)Apr 6 2023, 5:31 AM

Can you update the commit message to explain what OHOS is and provide a link to its homepage. That would be nice to have as a future reference.

Description is updated, please take a look

As usual: Do you plan to get official support for OpenHarmony in libc++? If that's the case, please take a look at https://libcxx.llvm.org/AddingNewCIJobs.html#addingnewcijobs.

Sure, we will try it, thank you

As usual: Do you plan to get official support for OpenHarmony in libc++? If that's the case, please take a look at https://libcxx.llvm.org/AddingNewCIJobs.html#addingnewcijobs.

Sure, we will try it, thank you

That would indeed be great to have.

LGTM, modulo one remark.

libcxx/include/__locale
46

At least I assume that was intended.

Mordante requested changes to this revision.Aug 31 2023, 10:36 AM

Friendly ping? We'll be moving to GitHub soon can you update the patch if still interested into getting this in libc++?

This revision now requires changes to proceed.Aug 31 2023, 10:36 AM

As usual: Do you plan to get official support for OpenHarmony in libc++? If that's the case, please take a look at https://libcxx.llvm.org/AddingNewCIJobs.html#addingnewcijobs.

+1 here, I don't think we can merge this unless we have a CI setup for it.

Friendly ping? We'll be moving to GitHub soon can you update the patch if still interested into getting this in libc++?

Sure, I am interested in it ) But to setup the CI for it, I need to get a dedicated server and setup runner there. I think it may take some time, atm I have no experience with Buildkite.
Hope I will be able to make it happen in this month

ldionne commandeered this revision.Nov 3 2023, 8:28 AM
ldionne edited reviewers, added: kpdev42; removed: ldionne.

I am going to commandeer this patch and abandon it in order to clear up our Phabricator queue. If you would like to pursue this patch, please re-open it on GitHub (and if possible mention this review!) and we'll be happy to review it. Thanks!

ldionne abandoned this revision.Nov 3 2023, 8:28 AM