Skip to content
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

2.2.4 failed test gdimagegrayscale/basic #359

Closed
remicollet opened this issue Jan 18, 2017 · 11 comments
Closed

2.2.4 failed test gdimagegrayscale/basic #359

remicollet opened this issue Jan 18, 2017 · 11 comments

Comments

@remicollet
Copy link
Contributor

Only on i386 (x86_64 is OK)

@remicollet
Copy link
Contributor Author

remicollet commented Jan 18, 2017

Diff:
basic c_23_diff

Out:
basic c_23_out

@cmb69
Copy link
Contributor

cmb69 commented Jan 18, 2017

Might be a rounding issue. I'll try to have a closer look at this.

@remicollet
Copy link
Contributor Author

Perhaps we could consider the result as acceptable....

@cmb69
Copy link
Contributor

cmb69 commented Jan 18, 2017

Perhaps we could consider the result as acceptable....

Indeed, but we'd need some tool for similarity comparison. See issue #216. I'm not completely against adding pdiff, but we may consider alternatives (SSIM comes to mind).

@remicollet
Copy link
Contributor Author

Notice: also fails on ppc64le, ppc64 and aarch64

@remicollet
Copy link
Contributor Author

Other arch fixed by b7b66ea (don't know why this wasn't applied in 2.2...), so only i686 affected

@cmb69
Copy link
Contributor

cmb69 commented Aug 9, 2017

@remicollet Have you tried with the custom CFLAGS mentioned in #242 (comment)?

@fweimer
Copy link

fweimer commented Jan 17, 2019

The test also fails for me on x86-64, with GCC 8.2 and -O2 -march=haswell -ftree-vectorize. Similar output difference. gdimagecopyresampled/bug00201 is also affected.

@J0WI
Copy link

J0WI commented Mar 4, 2019

s390x is also affected.

@kaniini
Copy link

kaniini commented Apr 3, 2020

mips64 is also affected

@pierrejoye
Copy link
Contributor

Works on most other 32bit builds, including windows VC.

mingw fails, could be solve using other FP flags.

As mentionned in the related tickets, we won't change the stable code for 32bit cygwin or mingw.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants