Hi!

Okay, another try:

olaf@x1:/store/olaf/workspace2/AspectC++$ svn commit -m "A third try to fix the problems on systems with g++ 13. Now the types _Float<N> are defined as destinct type s so that template specialization should work without conflicts. Furthermore, all types are now defined on all platforms, regardless of the availability of __float12
8."
Sende              NamespaceAC.cc
Übertrage Daten .erledigt
Übertrage Transaktion...
Revision 1667 übertragen.

I hope that this works now. The "fix" is actually just a workaround, because the types that I have created to satisfy the clang parser are not 100% compatible with the real _Float<N> types. So, it is possible that further improvements are needed.

Cheers,
Olaf

On 25.07.23 16:58, Olaf Spinczyk wrote:
Right.

/usr/include/math.h does not include a template specialization of struct __iseqsig_type for _Float<N> on my Ubuntu 22.04/AMD64 machine. Therefore, my local test had passed.

However, my workaround, which tries to satisfy the internal clang parser with typedefs, doesn't work in combination with this template specialization.

On Arm it is a different problem.

I'll try some other approach and keep you informed.

Cheers,

Olaf

On 25.07.23 12:49, Reinhard Tartler wrote:
Unfortunately, it made it worse :-(

Here is abuildlog for amd64: https://salsa.debian.org/debian/aspectc/-/jobs/4471659/raw

I've also attached my buildlog from an ARM64 machine to this email.

thanks for looking into this!

On 7/24/23 10:39 AM, Olaf Spinczyk wrote:
Hi!

It seems that my previous commit was too simple. I hope that today's modification helps:

olaf@x1:~/workspace2/AspectC++$ svn commit -m "Improved previous bugfix: Now the typedefs for _Float128 and _Float64x, which should make the clang parser happy, are
only generated on platforms where clang support __float128."
Sende              NamespaceAC.cc
Übertrage Daten .erledigt
Übertrage Transaktion...
Revision 1666 übertragen.

Cheers,
Olaf

On 24.07.23 13:26, Reinhard Tartler wrote:


On 7/23/23 9:13 AM, Reinhard Tartler wrote:

Turns out, the fix only works on amd64/i386, but not for instance on Arm, Matthias, our GCC maintainer has filed https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1041040 for this.

I looked over the patch and couldn't find anything architecture specific. Can you make any rhyme or reason for why it might fail on arm64?

Here are some more test logs:
https://ci.debian.net/packages/a/aspectc++/unstable/arm64/

This issue also occurs during a regular package build, not only during autopkgtest. I could reproduce it on the amdhal.debian.org in a sid chroot and got this:

Weaving aspects into PreFileIncluder.cc...
Weaving aspects into PreprocessorParser.cc...
Weaving aspects into UnitManager.cc...
In file included from /tmp/autopkgtest.sSXQhw/build.5EQ/src/Puma/gen-release/step1/src/PreFileIncluder.cc:19: In file included from /tmp/autopkgtest.sSXQhw/build.5EQ/src/Puma/gen-release/step1/inc/Puma/TokenStream.h:25: In file included from /tmp/autopkgtest.sSXQhw/build.5EQ/src/Puma/gen-release/step1/inc/Puma/Array.h:27: /tmp/autopkgtest.sSXQhw/build.5EQ/src/Puma/gen-release/step1/inc/Puma/Array.h_0_virt:66:9: error: __float128 is not supported on this target
typedef __float128 _Float128;
        ^
In file included from /tmp/autopkgtest.sSXQhw/build.5EQ/src/Puma/gen-release/step1/src/PrePrintVisitor.cc:19: In file included from /tmp/autopkgtest.sSXQhw/build.5EQ/src/Puma/gen-release/step1/inc/Puma/PreSemIterator.h:24: In file included from /tmp/autopkgtest.sSXQhw/build.5EQ/src/Puma/gen-release/step1/inc/Puma/PreTreeIterator.h:24: In file included from /tmp/autopkgtest.sSXQhw/build.5EQ/src/Puma/gen-release/step1/inc/Puma/PreTree.h:25: In file included from /tmp/autopkgtest.sSXQhw/build.5EQ/src/Puma/gen-release/step1/inc/Puma/PreVisitor.h:24: /tmp/autopkgtest.sSXQhw/build.5EQ/src/Puma/gen-release/step1/inc/Puma/PreVisitor.h_0_virt:66:9: error: __float128 is not supported on this target
typedef __float128 _Float128;
        ^
In file included from /tmp/autopkgtest.sSXQhw/build.5EQ/src/Puma/gen-release/step1/src/PreprocessorParser.cc:19: In file included from /tmp/autopkgtest.sSXQhw/build.5EQ/src/Puma/gen-release/step1/inc/Puma/Token.h:25: In file included from /tmp/autopkgtest.sSXQhw/build.5EQ/src/Puma/gen-release/step1/inc/Puma/LanguageID.h:25: /tmp/autopkgtest.sSXQhw/build.5EQ/src/Puma/gen-release/step1/inc/Puma/LanguageID.h_0_virt:66:9: error: __float128 is not supported on this target
typedef __float128 _Float128;
        ^
In file included from /tmp/autopkgtest.sSXQhw/build.5EQ/src/Puma/gen-release/step1/src/UnitManager.cc:19: In file included from /tmp/autopkgtest.sSXQhw/build.5EQ/src/Puma/gen-release/step1/inc/Puma/Unit.h:25: In file included from /tmp/autopkgtest.sSXQhw/build.5EQ/src/Puma/gen-release/step1/inc/Puma/List.h:25: In file included from /tmp/autopkgtest.sSXQhw/build.5EQ/src/Puma/gen-release/step1/inc/Puma/ListElement.h:27: /tmp/autopkgtest.sSXQhw/build.5EQ/src/Puma/gen-release/step1/inc/Puma/ListElement.h_0_virt:66:9: error: __float128 is not supported on this target
typedef __float128 _Float128;
        ^

_______________________________________________
aspectc-developers mailing list
aspectc-develop...@mail.aspectc.org
https://www.aspectc.org/cgi-bin/mailman/listinfo/aspectc-developers

_______________________________________________
aspectc-developers mailing list
aspectc-develop...@mail.aspectc.org
https://www.aspectc.org/cgi-bin/mailman/listinfo/aspectc-developers
_______________________________________________
aspectc-developers mailing list
aspectc-develop...@mail.aspectc.org
https://www.aspectc.org/cgi-bin/mailman/listinfo/aspectc-developers

Reply via email to