-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
open-amp: enable C library cause open-amp\open-amp\lib\common\sh_mem.c compile error #8104
Comments
When C library are you using? Which toolchain? |
this issue only happen when i enabled openamp function. if i disabled openamp function, c lib works good.@nashif |
I think this might get fixed when we update OpenAMP to the new version. See PR #11355 |
Hi, @galak, I have tested your branch (galak:openamp-v2018.10), and the question still exist, i have add <stdint.h> to <stdatomic.h>, but other errors occur, our product need <math.h>, so we must enable newlib support, hope your relay. |
Which c-library, how can I reproduce this? |
@galak newlib through menuconfig |
@galak |
@Jeepgoing what errors are you seeing when you do this with current tree? |
@galak there are something undefined errors from SDK |
can you post the errors you see currently, want to make sure i'm seeing the same thing. |
@galak my errors as title described, now is chinese new year, i don't have PC on my home, sorry. |
@galak Hi, thank you for your relay, do you have replicated this issue? |
@Jeepgoing if you can add a log of the failure you are seeing to the issue that would be great. |
@galak this is my compile error |
@Jeepgoing can you put the full build log. Also have you tried the 0.10.0-rc3 sdk? |
full log as below, and i have tried 0.10.0-rc3 sdk, it is the same error.
|
Please see libmetal patches: OpenAMP/libmetal#85 |
@Jeepgoing should hopefully now be fixed with PR #14203. |
@galak , ok, it's work, thanks. |
i am designing application program. when i enabled C Library, open-amp library compile error, as below:
The text was updated successfully, but these errors were encountered: