Message ID | 20230813172524.354647-1-giulio.benetti@benettiengineering.com |
---|---|
State | Accepted |
Headers | show |
Series | package/ramspeed: disable package if affected from gcc bug 43744 | expand |
On Sun, 13 Aug 2023 19:25:24 +0200 Giulio Benetti <giulio.benetti@benettiengineering.com> wrote: > This package is affected by gcc bug 43744 and I have not found a work > around for it(i.e. the common -O0 we use or other), so let's disable it if > gcc has such bug. > > Fixes: > http://autobuild.buildroot.net/results/ab289769c5fea435934ed260d38e0a4fdd2ba72d > > Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com> > --- > package/ramspeed/Config.in | 4 ++++ > 1 file changed, 4 insertions(+) Applied to master, thanks. FYI, gcc bug 43744 is marked as RESOLVED FIXED in the gcc bug tracker (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=43744), so if you don't reopen it, nobody is ever going to look into it :-/ Thomas
On 22/08/23 22:09, Thomas Petazzoni via buildroot wrote: > On Sun, 13 Aug 2023 19:25:24 +0200 > Giulio Benetti <giulio.benetti@benettiengineering.com> wrote: > >> This package is affected by gcc bug 43744 and I have not found a work >> around for it(i.e. the common -O0 we use or other), so let's disable it if >> gcc has such bug. >> >> Fixes: >> http://autobuild.buildroot.net/results/ab289769c5fea435934ed260d38e0a4fdd2ba72d >> >> Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com> >> --- >> package/ramspeed/Config.in | 4 ++++ >> 1 file changed, 4 insertions(+) > > Applied to master, thanks. FYI, gcc bug 43744 is marked as RESOLVED > FIXED in the gcc bug tracker > (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=43744), so if you don't > reopen it, nobody is ever going to look into it :-/ Thank you for pointing. Just commented and asked to reopen it. Best regards
>>>>> "Giulio" == Giulio Benetti <giulio.benetti@benettiengineering.com> writes: > This package is affected by gcc bug 43744 and I have not found a work > around for it(i.e. the common -O0 we use or other), so let's disable it if > gcc has such bug. > Fixes: > http://autobuild.buildroot.net/results/ab289769c5fea435934ed260d38e0a4fdd2ba72d > Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com> Committed to 2023.02.x and 2023.05.x, thanks.
diff --git a/package/ramspeed/Config.in b/package/ramspeed/Config.in index b236329a84..bd0438ffe1 100644 --- a/package/ramspeed/Config.in +++ b/package/ramspeed/Config.in @@ -1,5 +1,9 @@ +comment "ramspeed needs a toolchain not affected by GCC bug 43744" + depends on BR2_TOOLCHAIN_HAS_GCC_BUG_43744 + config BR2_PACKAGE_RAMSPEED bool "ramspeed" + depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_43744 help RAMspeed is a free open source command line utility to measure cache and memory performance.
This package is affected by gcc bug 43744 and I have not found a work around for it(i.e. the common -O0 we use or other), so let's disable it if gcc has such bug. Fixes: http://autobuild.buildroot.net/results/ab289769c5fea435934ed260d38e0a4fdd2ba72d Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com> --- package/ramspeed/Config.in | 4 ++++ 1 file changed, 4 insertions(+)