diff mbox series

[meta-oe,RFC] include-what-you-use: new recipe

Message ID 20240715121744.18888-1-ceggers@arri.de
State New
Headers show
Series [meta-oe,RFC] include-what-you-use: new recipe | expand

Commit Message

Christian Eggers July 15, 2024, 12:17 p.m. UTC
Based on proposal send previously here:
https://lists.yoctoproject.org/g/yocto/topic/recipe_for/83279588

This recipe depends on meta-clang, so should it be added to that layer?

Unfortunately the 'include-what-you-use' (iwyu) tool doesn't work yet.
Although I have installed nativesdk-clang (via CLANGSDK = "1") and
nativesdk-include-what-you-use packages, the C++ standard headers are
not found:

include-what-you-use --sysroot=/opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi impl/Fdt.cpp
impl/Fdt.cpp:9:10: fatal error: 'cerrno' file not found
    9 | #include <cerrno>
      |          ^~~~~~~~

As iwyu uses Clang and I have no experience working with it, this is
probably only a beginner mistake. Can Clang use the C++ headers provided
by libstdc++, or do I need to install further packages?

Signed-off-by: Christian Eggers <ceggers@arri.de>
---
 .../include-what-you-use_git.bb               | 22 +++++++++++++++++++
 1 file changed, 22 insertions(+)
 create mode 100644 meta-oe/recipes-devtools/include-what-you-use/include-what-you-use_git.bb

Comments

Khem Raj July 15, 2024, 2:18 p.m. UTC | #1
On Mon, Jul 15, 2024 at 5:18 AM Christian Eggers via lists.openembedded.org
<ceggers=arri.de@lists.openembedded.org> wrote:

> Based on proposal send previously here:
> https://lists.yoctoproject.org/g/yocto/topic/recipe_for/83279588
>
> This recipe depends on meta-clang, so should it be added to that layer?
>
> Unfortunately the 'include-what-you-use' (iwyu) tool doesn't work yet.
> Although I have installed nativesdk-clang (via CLANGSDK = "1") and
> nativesdk-include-what-you-use packages, the C++ standard headers are
> not found:
>
> include-what-you-use
> --sysroot=/opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi
> impl/Fdt.cpp
> impl/Fdt.cpp:9:10: fatal error: 'cerrno' file not found
>     9 | #include <cerrno>
>       |          ^~~~~~~~
>
> As iwyu uses Clang and I have no experience working with it, this is
> probably only a beginner mistake. Can Clang use the C++ headers provided
> by libstdc++, or do I need to install further packages?
>

It can but it seems your SDK install might be missing any c++ runtime so
check a simple program that includes say <iostream> header and try
compiling it


> Signed-off-by: Christian Eggers <ceggers@arri.de>
> ---
>  .../include-what-you-use_git.bb               | 22 +++++++++++++++++++
>  1 file changed, 22 insertions(+)
>  create mode 100644 meta-oe/recipes-devtools/include-what-you-use/
> include-what-you-use_git.bb
>
> diff --git a/meta-oe/recipes-devtools/include-what-you-use/
> include-what-you-use_git.bb
> b/meta-oe/recipes-devtools/include-what-you-use/
> include-what-you-use_git.bb
> new file mode 100644
> index 000000000000..e7df6a834a34
> --- /dev/null
> +++ b/meta-oe/recipes-devtools/include-what-you-use/
> include-what-you-use_git.bb
> @@ -0,0 +1,22 @@
> +LICENSE = "NCSA"
> +LIC_FILES_CHKSUM =
> "file://LICENSE.TXT;md5=59d01ad98720f3c50d6a8a0ef3108c88 \
> +
> file://iwyu-check-license-header.py;md5=7bdb749831163fbe9232b3cb7186116f"
> +
> +SRC_URI = "git://
> github.com/include-what-you-use/include-what-you-use.git;protocol=https;branch=clang_18
> "
> +
> +PV = "0.22+git${SRCPV}"
> +SRCREV = "377eaef70cdda47368939f4d9beabfabe3f628f0"
> +
> +S = "${WORKDIR}/git"
> +
> +DEPENDS = "clang"


This means it needs to use clang as library to link to is that what it
required or does it need clang as static compiler to build this package in
which case you can do so by using TOOLCHAIN = “clang” in recipe


> +
> +inherit cmake python3native
> +
> +EXTRA_OECMAKE:append:class-nativesdk = " \
> +       -DCMAKE_SKIP_RPATH:BOOL=ON \
> +       "
> +
> +FILES:${PN} += "${datadir}/${BPN}"
> +
> +BBCLASSEXTEND = "nativesdk"
> --
> 2.44.1
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#111360):
> https://lists.openembedded.org/g/openembedded-devel/message/111360
> Mute This Topic: https://lists.openembedded.org/mt/107229738/1997914
> Group Owner: openembedded-devel+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [
> raj.khem@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>
Christian Eggers July 16, 2024, 8:08 a.m. UTC | #2
On Monday, 15 July 2024, 16:18:14 CEST, Khem Raj wrote:
> On Mon, Jul 15, 2024 at 5:18 AM Christian Eggers via lists.openembedded.org
> <ceggers=arri.de@lists.openembedded.org> wrote:
> 
> > Based on proposal send previously here:
> > https://lists.yoctoproject.org/g/yocto/topic/recipe_for/83279588
> >
> > This recipe depends on meta-clang, so should it be added to that layer?
> >
> > Unfortunately the 'include-what-you-use' (iwyu) tool doesn't work yet.
> > Although I have installed nativesdk-clang (via CLANGSDK = "1") and
> > nativesdk-include-what-you-use packages, the C++ standard headers are
> > not found:
> >
> > include-what-you-use
> > --sysroot=/opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi
> > impl/Fdt.cpp
> > impl/Fdt.cpp:9:10: fatal error: 'cerrno' file not found
> >     9 | #include <cerrno>
> >       |          ^~~~~~~~
> >
> > As iwyu uses Clang and I have no experience working with it, this is
> > probably only a beginner mistake. Can Clang use the C++ headers provided
> > by libstdc++, or do I need to install further packages?
> >
> 
> It can but it seems your SDK install might be missing any c++ runtime so
> check a simple program that includes say <iostream> header and try
> compiling it
My SDK definitely contains std:: C++ headers for target and nativesdk. But the
libstdc++ C++ headers are installed in GCC specific locations:

/opt/poky/5.0.2/sysroots/x86_64-pokysdk-linux/usr/include/c++/13.3.0/
/opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi/usr/include/c++/13.3.0

How can Clang know that the C++ headers are located in the (GCC specific) c++/13.3.0
sub directory?


> 
> 
> > Signed-off-by: Christian Eggers <ceggers@arri.de>
> > ---
> >  .../include-what-you-use_git.bb               | 22 +++++++++++++++++++
> >  1 file changed, 22 insertions(+)
> >  create mode 100644 meta-oe/recipes-devtools/include-what-you-use/
> > include-what-you-use_git.bb
> >
> > diff --git a/meta-oe/recipes-devtools/include-what-you-use/
> > include-what-you-use_git.bb
> > b/meta-oe/recipes-devtools/include-what-you-use/
> > include-what-you-use_git.bb
> > new file mode 100644
> > index 000000000000..e7df6a834a34
> > --- /dev/null
> > +++ b/meta-oe/recipes-devtools/include-what-you-use/
> > include-what-you-use_git.bb
> > @@ -0,0 +1,22 @@
> > +LICENSE = "NCSA"
> > +LIC_FILES_CHKSUM =
> > "file://LICENSE.TXT;md5=59d01ad98720f3c50d6a8a0ef3108c88 \
> > +
> > file://iwyu-check-license-header.py;md5=7bdb749831163fbe9232b3cb7186116f"
> > +
> > +SRC_URI = "git://
> > github.com/include-what-you-use/include-what-you-use.git;protocol=https;branch=clang_18
> > "
> > +
> > +PV = "0.22+git${SRCPV}"
> > +SRCREV = "377eaef70cdda47368939f4d9beabfabe3f628f0"
> > +
> > +S = "${WORKDIR}/git"
> > +
> > +DEPENDS = "clang"
> 
> 
> This means it needs to use clang as library to link to is that what it
> required or does it need clang as static compiler to build this package in
> which case you can do so by using TOOLCHAIN = “clang” in recipe
I think that it builds fine with the GCC toolchain, at least it doesn't
complain during build. Finally it links against libclang-cpp.so.18.1 and 
libLLVM.so.18.1, so using "DEPENDS" sounds correct for me.

Although I haven't configured Clang to provide also llvm (as described here
https://github.com/kraj/meta-clang?tab=readme-ov-file#providing-llvm),
there is a nativesdk-clang-libllvm package after building nativesdk-clang.
> 
> 
> > +
> > +inherit cmake python3native
> > +
> > +EXTRA_OECMAKE:append:class-nativesdk = " \
> > +       -DCMAKE_SKIP_RPATH:BOOL=ON \
> > +       "
> > +
> > +FILES:${PN} += "${datadir}/${BPN}"
> > +
> > +BBCLASSEXTEND = "nativesdk"
> > --
> > 2.44.1
> >
> >
> > -=-=-=-=-=-=-=-=-=-=-=-
> > Links: You receive all messages sent to this group.
> > View/Reply Online (#111360):
> > https://lists.openembedded.org/g/openembedded-devel/message/111360
> > Mute This Topic: https://lists.openembedded.org/mt/107229738/1997914
> > Group Owner: openembedded-devel+owner@lists.openembedded.org
> > Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [
> > raj.khem@gmail.com]
> > -=-=-=-=-=-=-=-=-=-=-=-
> >
> >
>
Khem Raj July 16, 2024, 1:45 p.m. UTC | #3
On Tue, Jul 16, 2024 at 1:08 AM Christian Eggers <ceggers@arri.de> wrote:

> On Monday, 15 July 2024, 16:18:14 CEST, Khem Raj wrote:
> > On Mon, Jul 15, 2024 at 5:18 AM Christian Eggers via
> lists.openembedded.org
> > <ceggers=arri.de@lists.openembedded.org> wrote:
> >
> > > Based on proposal send previously here:
> > > https://lists.yoctoproject.org/g/yocto/topic/recipe_for/83279588
> > >
> > > This recipe depends on meta-clang, so should it be added to that layer?
> > >
> > > Unfortunately the 'include-what-you-use' (iwyu) tool doesn't work yet.
> > > Although I have installed nativesdk-clang (via CLANGSDK = "1") and
> > > nativesdk-include-what-you-use packages, the C++ standard headers are
> > > not found:
> > >
> > > include-what-you-use
> > >
> --sysroot=/opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi
> > > impl/Fdt.cpp
> > > impl/Fdt.cpp:9:10: fatal error: 'cerrno' file not found
> > >     9 | #include <cerrno>
> > >       |          ^~~~~~~~
> > >
> > > As iwyu uses Clang and I have no experience working with it, this is
> > > probably only a beginner mistake. Can Clang use the C++ headers
> provided
> > > by libstdc++, or do I need to install further packages?
> > >
> >
> > It can but it seems your SDK install might be missing any c++ runtime so
> > check a simple program that includes say <iostream> header and try
> > compiling it
> My SDK definitely contains std:: C++ headers for target and nativesdk. But
> the
> libstdc++ C++ headers are installed in GCC specific locations:
>
> /opt/poky/5.0.2/sysroots/x86_64-pokysdk-linux/usr/include/c++/13.3.0/
>
> /opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi/usr/include/c++/13.3.0
>
> How can Clang know that the C++ headers are located in the (GCC specific)
> c++/13.3.0
> sub directory?


Try adding -stdlib=libstdc++ on clang cmdline

>
>
>
> >
> >
> > > Signed-off-by: Christian Eggers <ceggers@arri.de>
> > > ---
> > >  .../include-what-you-use_git.bb               | 22
> +++++++++++++++++++
> > >  1 file changed, 22 insertions(+)
> > >  create mode 100644 meta-oe/recipes-devtools/include-what-you-use/
> > > include-what-you-use_git.bb
> > >
> > > diff --git a/meta-oe/recipes-devtools/include-what-you-use/
> > > include-what-you-use_git.bb
> > > b/meta-oe/recipes-devtools/include-what-you-use/
> > > include-what-you-use_git.bb
> > > new file mode 100644
> > > index 000000000000..e7df6a834a34
> > > --- /dev/null
> > > +++ b/meta-oe/recipes-devtools/include-what-you-use/
> > > include-what-you-use_git.bb
> > > @@ -0,0 +1,22 @@
> > > +LICENSE = "NCSA"
> > > +LIC_FILES_CHKSUM =
> > > "file://LICENSE.TXT;md5=59d01ad98720f3c50d6a8a0ef3108c88 \
> > > +
> > >
> file://iwyu-check-license-header.py;md5=7bdb749831163fbe9232b3cb7186116f"
> > > +
> > > +SRC_URI = "git://
> > >
> github.com/include-what-you-use/include-what-you-use.git;protocol=https;branch=clang_18
> > > "
> > > +
> > > +PV = "0.22+git${SRCPV}"
> > > +SRCREV = "377eaef70cdda47368939f4d9beabfabe3f628f0"
> > > +
> > > +S = "${WORKDIR}/git"
> > > +
> > > +DEPENDS = "clang"
> >
> >
> > This means it needs to use clang as library to link to is that what it
> > required or does it need clang as static compiler to build this package
> in
> > which case you can do so by using TOOLCHAIN = “clang” in recipe
> I think that it builds fine with the GCC toolchain, at least it doesn't
> complain during build. Finally it links against libclang-cpp.so.18.1 and
> libLLVM.so.18.1, so using "DEPENDS" sounds correct for me.
>
> Although I haven't configured Clang to provide also llvm (as described here
> https://github.com/kraj/meta-clang?tab=readme-ov-file#providing-llvm),
> there is a nativesdk-clang-libllvm package after building nativesdk-clang.
> >
> >
> > > +
> > > +inherit cmake python3native
> > > +
> > > +EXTRA_OECMAKE:append:class-nativesdk = " \
> > > +       -DCMAKE_SKIP_RPATH:BOOL=ON \
> > > +       "
> > > +
> > > +FILES:${PN} += "${datadir}/${BPN}"
> > > +
> > > +BBCLASSEXTEND = "nativesdk"
> > > --
> > > 2.44.1
> > >
> > >
> > > -=-=-=-=-=-=-=-=-=-=-=-
> > > Links: You receive all messages sent to this group.
> > > View/Reply Online (#111360):
> > > https://lists.openembedded.org/g/openembedded-devel/message/111360
> > > Mute This Topic: https://lists.openembedded.org/mt/107229738/1997914
> > > Group Owner: openembedded-devel+owner@lists.openembedded.org
> > > Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub
> [
> > > raj.khem@gmail.com]
> > > -=-=-=-=-=-=-=-=-=-=-=-
> > >
> > >
> >
>
>
>
>
>
Christian Eggers July 16, 2024, 2:57 p.m. UTC | #4
Hi Khem,


On Tuesday, 16 July 2024, 15:45:54 CEST, Khem Raj wrote:
> On Tue, Jul 16, 2024 at 1:08 AM Christian Eggers <ceggers@arri.de> wrote:
> 
> > On Monday, 15 July 2024, 16:18:14 CEST, Khem Raj wrote:
> > > On Mon, Jul 15, 2024 at 5:18 AM Christian Eggers via
> > lists.openembedded.org
> > > <ceggers=arri.de@lists.openembedded.org> wrote:
> > >
> > > > Based on proposal send previously here:
> > > > https://lists.yoctoproject.org/g/yocto/topic/recipe_for/83279588
> > > >
> > > > This recipe depends on meta-clang, so should it be added to that layer?
> > > >
> > > > Unfortunately the 'include-what-you-use' (iwyu) tool doesn't work yet.
> > > > Although I have installed nativesdk-clang (via CLANGSDK = "1") and
> > > > nativesdk-include-what-you-use packages, the C++ standard headers are
> > > > not found:
> > > >
> > > > include-what-you-use
> > > >
> > --sysroot=/opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi
> > > > impl/Fdt.cpp
> > > > impl/Fdt.cpp:9:10: fatal error: 'cerrno' file not found
> > > >     9 | #include <cerrno>
> > > >       |          ^~~~~~~~
> > > >
> > > > As iwyu uses Clang and I have no experience working with it, this is
> > > > probably only a beginner mistake. Can Clang use the C++ headers
> > provided
> > > > by libstdc++, or do I need to install further packages?
> > > >
> > >
> > > It can but it seems your SDK install might be missing any c++ runtime so
> > > check a simple program that includes say <iostream> header and try
> > > compiling it
> > My SDK definitely contains std:: C++ headers for target and nativesdk. But
> > the
> > libstdc++ C++ headers are installed in GCC specific locations:
> >
> > /opt/poky/5.0.2/sysroots/x86_64-pokysdk-linux/usr/include/c++/13.3.0/
> >
> > /opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi/usr/include/c++/13.3.0
> >
> > How can Clang know that the C++ headers are located in the (GCC specific)
> > c++/13.3.0
> > sub directory?
> 
> 
> Try adding -stdlib=libstdc++ on clang cmdline

The following seems to work:
include-what-you-use \
    -target arm-poky-linux-musleabi \
    --sysroot /opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi \
    -stdlib++-isystem /opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi/usr/include/c++/13.3.0/arm-poky-linux-musleabi \
    -stdlib++-isystem /opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi/usr/include/c++/13.3.0  \
    File.cpp

Is there a way to make this a little bit handier? I would like to avoid hard coding this into my CMake toolchain file.

regards,
Christian
Khem Raj July 16, 2024, 3:42 p.m. UTC | #5
On Tue, Jul 16, 2024 at 7:57 AM Christian Eggers <ceggers@arri.de> wrote:
>
> Hi Khem,
>
>
> On Tuesday, 16 July 2024, 15:45:54 CEST, Khem Raj wrote:
> > On Tue, Jul 16, 2024 at 1:08 AM Christian Eggers <ceggers@arri.de> wrote:
> >
> > > On Monday, 15 July 2024, 16:18:14 CEST, Khem Raj wrote:
> > > > On Mon, Jul 15, 2024 at 5:18 AM Christian Eggers via
> > > lists.openembedded.org
> > > > <ceggers=arri.de@lists.openembedded.org> wrote:
> > > >
> > > > > Based on proposal send previously here:
> > > > > https://lists.yoctoproject.org/g/yocto/topic/recipe_for/83279588
> > > > >
> > > > > This recipe depends on meta-clang, so should it be added to that layer?
> > > > >
> > > > > Unfortunately the 'include-what-you-use' (iwyu) tool doesn't work yet.
> > > > > Although I have installed nativesdk-clang (via CLANGSDK = "1") and
> > > > > nativesdk-include-what-you-use packages, the C++ standard headers are
> > > > > not found:
> > > > >
> > > > > include-what-you-use
> > > > >
> > > --sysroot=/opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi
> > > > > impl/Fdt.cpp
> > > > > impl/Fdt.cpp:9:10: fatal error: 'cerrno' file not found
> > > > >     9 | #include <cerrno>
> > > > >       |          ^~~~~~~~
> > > > >
> > > > > As iwyu uses Clang and I have no experience working with it, this is
> > > > > probably only a beginner mistake. Can Clang use the C++ headers
> > > provided
> > > > > by libstdc++, or do I need to install further packages?
> > > > >
> > > >
> > > > It can but it seems your SDK install might be missing any c++ runtime so
> > > > check a simple program that includes say <iostream> header and try
> > > > compiling it
> > > My SDK definitely contains std:: C++ headers for target and nativesdk. But
> > > the
> > > libstdc++ C++ headers are installed in GCC specific locations:
> > >
> > > /opt/poky/5.0.2/sysroots/x86_64-pokysdk-linux/usr/include/c++/13.3.0/
> > >
> > > /opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi/usr/include/c++/13.3.0
> > >
> > > How can Clang know that the C++ headers are located in the (GCC specific)
> > > c++/13.3.0
> > > sub directory?
> >
> >
> > Try adding -stdlib=libstdc++ on clang cmdline
>
> The following seems to work:
> include-what-you-use \
>     -target arm-poky-linux-musleabi \
>     --sysroot /opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi \
>     -stdlib++-isystem /opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi/usr/include/c++/13.3.0/arm-poky-linux-musleabi \
>     -stdlib++-isystem /opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi/usr/include/c++/13.3.0  \
>     File.cpp
>
> Is there a way to make this a little bit handier? I would like to avoid hard coding this into my CMake toolchain file.
>

Hmm so -stdlib=libstdc++ does not work ?

> regards,
> Christian
>
>
>
Christian Eggers July 19, 2024, 2:10 p.m. UTC | #6
Hi Khem,

sorry for the delay.

On Tuesday, 16 July 2024, 17:42:33 CEST, Khem Raj wrote:
> On Tue, Jul 16, 2024 at 7:57 AM Christian Eggers <ceggers@arri.de> wrote:
> >
> > Hi Khem,
> >
> >
> > On Tuesday, 16 July 2024, 15:45:54 CEST, Khem Raj wrote:
> > > On Tue, Jul 16, 2024 at 1:08 AM Christian Eggers <ceggers@arri.de> wrote:
> > >
> > > > On Monday, 15 July 2024, 16:18:14 CEST, Khem Raj wrote:
> > > > > On Mon, Jul 15, 2024 at 5:18 AM Christian Eggers via
> > > > lists.openembedded.org
> > > > > <ceggers=arri.de@lists.openembedded.org> wrote:
> > > > >
> > > > > > Based on proposal send previously here:
> > > > > > https://lists.yoctoproject.org/g/yocto/topic/recipe_for/83279588
> > > > > >
> > > > > > This recipe depends on meta-clang, so should it be added to that layer?
> > > > > >
> > > > > > Unfortunately the 'include-what-you-use' (iwyu) tool doesn't work yet.
> > > > > > Although I have installed nativesdk-clang (via CLANGSDK = "1") and
> > > > > > nativesdk-include-what-you-use packages, the C++ standard headers are
> > > > > > not found:
> > > > > >
> > > > > > include-what-you-use
> > > > > >
> > > > --sysroot=/opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi
> > > > > > impl/Fdt.cpp
> > > > > > impl/Fdt.cpp:9:10: fatal error: 'cerrno' file not found
> > > > > >     9 | #include <cerrno>
> > > > > >       |          ^~~~~~~~
> > > > > >
> > > > > > As iwyu uses Clang and I have no experience working with it, this is
> > > > > > probably only a beginner mistake. Can Clang use the C++ headers
> > > > provided
> > > > > > by libstdc++, or do I need to install further packages?
> > > > > >
> > > > >
> > > > > It can but it seems your SDK install might be missing any c++ runtime so
> > > > > check a simple program that includes say <iostream> header and try
> > > > > compiling it
> > > > My SDK definitely contains std:: C++ headers for target and nativesdk. But
> > > > the
> > > > libstdc++ C++ headers are installed in GCC specific locations:
> > > >
> > > > /opt/poky/5.0.2/sysroots/x86_64-pokysdk-linux/usr/include/c++/13.3.0/
> > > >
> > > > /opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi/usr/include/c++/13.3.0
> > > >
> > > > How can Clang know that the C++ headers are located in the (GCC specific)
> > > > c++/13.3.0
> > > > sub directory?
> > >
> > >
> > > Try adding -stdlib=libstdc++ on clang cmdline
> >
> > The following seems to work:
> > include-what-you-use \
> >     -target arm-poky-linux-musleabi \
> >     --sysroot /opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi \
> >     -stdlib++-isystem /opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi/usr/include/c++/13.3.0/arm-poky-linux-musleabi \
> >     -stdlib++-isystem /opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi/usr/include/c++/13.3.0  \
> >     File.cpp
> >
> > Is there a way to make this a little bit handier? I would like to avoid hard coding this into my CMake toolchain file.
> >
> 
> Hmm so -stdlib=libstdc++ does not work ?

I cannot reproduce what exactly went wrong, but now the following works fine:

include-what-you-use -target arm-poky-linux-musleabi --sysroot /opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi test.cpp
include-what-you-use -target x86_64-pokysdk-linux --sysroot /opt/orbiter-tiny/5.0.2/sysroots/x86_64-pokysdk-linux test.cpp

So neither '-stdlib=libstdc++' nor '-stdlib++-isystem' is required anymore.

I was also able to integrate this into a CMake project (built with the SDK) by adding the following to my toolchain file:

set(CMAKE_C_INCLUDE_WHAT_YOU_USE include-what-you-use;-target;arm-poky-linux-musleabi;--sysroot;$ENV{SDKTARGETSYSROOT})
set(CMAKE_CXX_INCLUDE_WHAT_YOU_USE include-what-you-use;-target;arm-poky-linux-musleabi;--sysroot;$ENV{SDKTARGETSYSROOT})

The IWYU tool states itself as "experimental software", so the output
better should be interpreted by a human. I've tested with an existing
code base and got many valuable hints for missing/superfluous
#include statements. The results are quite good for std:: C/C++
headers, but other libraries like Boost need further improvements.

Can this recipe be added to meta-clang (due to it's dependencies to libclang/libllvm)?
Khem Raj July 19, 2024, 3:57 p.m. UTC | #7
On Fri, Jul 19, 2024 at 7:10 AM Christian Eggers <ceggers@arri.de> wrote:

> Hi Khem,
>
> sorry for the delay.
>
> On Tuesday, 16 July 2024, 17:42:33 CEST, Khem Raj wrote:
> > On Tue, Jul 16, 2024 at 7:57 AM Christian Eggers <ceggers@arri.de>
> wrote:
> > >
> > > Hi Khem,
> > >
> > >
> > > On Tuesday, 16 July 2024, 15:45:54 CEST, Khem Raj wrote:
> > > > On Tue, Jul 16, 2024 at 1:08 AM Christian Eggers <ceggers@arri.de>
> wrote:
> > > >
> > > > > On Monday, 15 July 2024, 16:18:14 CEST, Khem Raj wrote:
> > > > > > On Mon, Jul 15, 2024 at 5:18 AM Christian Eggers via
> > > > > lists.openembedded.org
> > > > > > <ceggers=arri.de@lists.openembedded.org> wrote:
> > > > > >
> > > > > > > Based on proposal send previously here:
> > > > > > >
> https://lists.yoctoproject.org/g/yocto/topic/recipe_for/83279588
> > > > > > >
> > > > > > > This recipe depends on meta-clang, so should it be added to
> that layer?
> > > > > > >
> > > > > > > Unfortunately the 'include-what-you-use' (iwyu) tool doesn't
> work yet.
> > > > > > > Although I have installed nativesdk-clang (via CLANGSDK = "1")
> and
> > > > > > > nativesdk-include-what-you-use packages, the C++ standard
> headers are
> > > > > > > not found:
> > > > > > >
> > > > > > > include-what-you-use
> > > > > > >
> > > > >
> --sysroot=/opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi
> > > > > > > impl/Fdt.cpp
> > > > > > > impl/Fdt.cpp:9:10: fatal error: 'cerrno' file not found
> > > > > > >     9 | #include <cerrno>
> > > > > > >       |          ^~~~~~~~
> > > > > > >
> > > > > > > As iwyu uses Clang and I have no experience working with it,
> this is
> > > > > > > probably only a beginner mistake. Can Clang use the C++ headers
> > > > > provided
> > > > > > > by libstdc++, or do I need to install further packages?
> > > > > > >
> > > > > >
> > > > > > It can but it seems your SDK install might be missing any c++
> runtime so
> > > > > > check a simple program that includes say <iostream> header and
> try
> > > > > > compiling it
> > > > > My SDK definitely contains std:: C++ headers for target and
> nativesdk. But
> > > > > the
> > > > > libstdc++ C++ headers are installed in GCC specific locations:
> > > > >
> > > > >
> /opt/poky/5.0.2/sysroots/x86_64-pokysdk-linux/usr/include/c++/13.3.0/
> > > > >
> > > > >
> /opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi/usr/include/c++/13.3.0
> > > > >
> > > > > How can Clang know that the C++ headers are located in the (GCC
> specific)
> > > > > c++/13.3.0
> > > > > sub directory?
> > > >
> > > >
> > > > Try adding -stdlib=libstdc++ on clang cmdline
> > >
> > > The following seems to work:
> > > include-what-you-use \
> > >     -target arm-poky-linux-musleabi \
> > >     --sysroot
> /opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi \
> > >     -stdlib++-isystem
> /opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi/usr/include/c++/13.3.0/arm-poky-linux-musleabi
> \
> > >     -stdlib++-isystem
> /opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi/usr/include/c++/13.3.0
> \
> > >     File.cpp
> > >
> > > Is there a way to make this a little bit handier? I would like to
> avoid hard coding this into my CMake toolchain file.
> > >
> >
> > Hmm so -stdlib=libstdc++ does not work ?
>
> I cannot reproduce what exactly went wrong, but now the following works
> fine:
>
> include-what-you-use -target arm-poky-linux-musleabi --sysroot
> /opt/poky/5.0.2/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-musleabi
> test.cpp
> include-what-you-use -target x86_64-pokysdk-linux --sysroot
> /opt/orbiter-tiny/5.0.2/sysroots/x86_64-pokysdk-linux test.cpp
>
> So neither '-stdlib=libstdc++' nor '-stdlib++-isystem' is required anymore.
>
> I was also able to integrate this into a CMake project (built with the
> SDK) by adding the following to my toolchain file:
>
> set(CMAKE_C_INCLUDE_WHAT_YOU_USE
> include-what-you-use;-target;arm-poky-linux-musleabi;--sysroot;$ENV{SDKTARGETSYSROOT})
> set(CMAKE_CXX_INCLUDE_WHAT_YOU_USE
> include-what-you-use;-target;arm-poky-linux-musleabi;--sysroot;$ENV{SDKTARGETSYSROOT})
>
> The IWYU tool states itself as "experimental software", so the output
> better should be interpreted by a human. I've tested with an existing
> code base and got many valuable hints for missing/superfluous
> #include statements. The results are quite good for std:: C/C++
> headers, but other libraries like Boost need further improvements.
>
> Can this recipe be added to meta-clang (due to it's dependencies to
> libclang/libllvm)?
>
>
Yes send a pr to meta-clang

>
>
>
>
diff mbox series

Patch

diff --git a/meta-oe/recipes-devtools/include-what-you-use/include-what-you-use_git.bb b/meta-oe/recipes-devtools/include-what-you-use/include-what-you-use_git.bb
new file mode 100644
index 000000000000..e7df6a834a34
--- /dev/null
+++ b/meta-oe/recipes-devtools/include-what-you-use/include-what-you-use_git.bb
@@ -0,0 +1,22 @@ 
+LICENSE = "NCSA"
+LIC_FILES_CHKSUM = "file://LICENSE.TXT;md5=59d01ad98720f3c50d6a8a0ef3108c88 \
+                    file://iwyu-check-license-header.py;md5=7bdb749831163fbe9232b3cb7186116f"
+
+SRC_URI = "git://github.com/include-what-you-use/include-what-you-use.git;protocol=https;branch=clang_18"
+
+PV = "0.22+git${SRCPV}"
+SRCREV = "377eaef70cdda47368939f4d9beabfabe3f628f0"
+
+S = "${WORKDIR}/git"
+
+DEPENDS = "clang"
+
+inherit cmake python3native
+
+EXTRA_OECMAKE:append:class-nativesdk = " \
+	-DCMAKE_SKIP_RPATH:BOOL=ON \
+	"
+
+FILES:${PN} += "${datadir}/${BPN}"
+
+BBCLASSEXTEND = "nativesdk"