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

test-gtc_packing fails on i686 #212

Closed
muep opened this issue Jun 11, 2014 · 6 comments
Closed

test-gtc_packing fails on i686 #212

muep opened this issue Jun 11, 2014 · 6 comments

Comments

@muep
Copy link

muep commented Jun 11, 2014

Seems to happen with all 0.9.5* versions.

Steps to reproduce on GCC 4.8 on GNU/Linux, starting with PWD at the glm repository root

mkdir build
cd build
g++ -m32 -O2 -Wall -I .. ../test/gtc/gtc_packing.cpp
./a.out ; echo $?

The resulting test program exits with value 6, failing the test set.

So far, I have only had this happen on i686. Also it does not happen with -O1.

@muep
Copy link
Author

muep commented Jun 11, 2014

Just as an additional note, the issue does not seem to affect either of clang 3.4 of Fedora 20 and clang 3.4.1 of Fedora rawhide.

@Groovounet Groovounet added the bug label Jun 20, 2014
@Groovounet Groovounet added this to the GLM 0.9.5 milestone Jun 20, 2014
@Groovounet Groovounet self-assigned this Jun 20, 2014
Groovounet pushed a commit that referenced this issue Jun 21, 2014
Groovounet pushed a commit that referenced this issue Jun 21, 2014
@Groovounet
Copy link
Member

Hi,

I think I fixed the issue but I could not reproduce the bug.
Let me know if this is working for you now.

Thanks,
Christophe

@muep
Copy link
Author

muep commented Jun 23, 2014

Hello,

The problem still persists for me on i686, at least with commits 6f59e64 and 4fa38c7.

It is nice that the aliasing issues got fixed, though.

@Groovounet Groovounet reopened this Jun 23, 2014
@Groovounet
Copy link
Member

Damn, that bug is going to be a bummer. :/

@muep
Copy link
Author

muep commented Jan 6, 2015

Just a note that this issue is still present with GCC 4.9 as shipped in Fedora 21.

The issue does not appear if the -ffloat-store option is passed to GCC.

@Groovounet Groovounet removed this from the GLM 0.9.5 milestone Nov 1, 2016
@Groovounet Groovounet added this to the GLM 0.9.9 milestone Nov 20, 2016
@Groovounet
Copy link
Member

I finally figured the problem with this issue: Denorms!

This issue is fixed in master and 0.9.8 branches for GLM 0.9.8.4 release.

Thanks for reporting,
Christophe

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

2 participants