ARC: MWDT: add C++ support #37653
Merged
ARC: MWDT: add C++ support #37653
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
github-actions
bot
added
area: API
abrodkin
approved these changes
Aug 13, 2021
The MWDT toolchain provides two C++ libraries. Let's use LLVM C++ library. Signed-off-by: Eugeniy Paltsev <[email protected]> Signed-off-by: Evgeniy Paltsev <[email protected]>
As of today we have libraries provided by MWDT build with stackcheck enabled. So we have to provide dummy _fstack, _estack to make it working. Signed-off-by: Eugeniy Paltsev <[email protected]> Signed-off-by: Evgeniy Paltsev <[email protected]>
Add missing properties for C++14, C++17 standards support Signed-off-by: Eugeniy Paltsev <[email protected]> Signed-off-by: Evgeniy Paltsev <[email protected]>
evgeniy-paltsev
force-pushed
the
rff-mwdt-cpp-support-w-cpp-pr-v12
branch
from
45c9b4f to
fd43adf
Compare
Add cpp.libcxx.arcmwdtlib to test C++ library from MWDT toolchain (with MWDT C library) Signed-off-by: Eugeniy Paltsev <[email protected]> Signed-off-by: Evgeniy Paltsev <[email protected]>
Fix include path of headers provided by MWDT toolchain for the case of MWDT C/C++ lib usage. Signed-off-by: Eugeniy Paltsev <[email protected]> Signed-off-by: Evgeniy Paltsev <[email protected]>
evgeniy-paltsev
force-pushed
the
rff-mwdt-cpp-support-w-cpp-pr-v12
branch
from
fd43adf to
906211d
Compare
ARC MWDT doesn't support building Zephyr as an native application (CONFIG_NATIVE_APPLICATION). ARC MWDT doesn't support building with CONFIG_NEWLIB_LIBC as it doesn't have newlib. Let's explicitly forbid to use these Kconfig options for the MWDT toolchain. Signed-off-by: Eugeniy Paltsev <[email protected]> Signed-off-by: Evgeniy Paltsev <[email protected]>
evgeniy-paltsev
force-pushed
the
rff-mwdt-cpp-support-w-cpp-pr-v12
branch
from
906211d to
f33e52d
Compare
evgeniy-paltsev
changed the title
evgeniy-paltsev
requested review from
andyross,
dcpleung,
nashif,
ruuddw and
tejlmand
as code owners
ruuddw
approved these changes
Aug 19, 2021
|
@evgeniy-paltsev - is this ready to merge? |
alexanderwachter
approved these changes
Aug 21, 2021
Yes, it's ready. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add C++ libs support when building with MWDT toolchain.