search

harrison-lucas / bullet

Automatically exported from code.google.com/p/bullet
Other
0 stars 0 forks source link

iOS Neon support for 64bit #739

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. The Neon implementation uses a 32bit binary code which does not compile in 
64bit

What version of the product are you using? On what operating system?
r2672

Please provide any additional information below.
Line number 887 in btVector3.cpp has 32bit asm code. We need to identify the 
compiler and support 64bit version as well.

Original issue reported on code.google.com by nharisha...@gmail.com on 17 Sep 2013 at 4:30

GoogleCodeExporter commented 9 years ago 
Can you detect the 64bit version and disable the assembly for now?

Are one of the following preprocessor definitions defined in LLVM 64bit?

#if defined(__LP64__) || defined(_LP64)
//disable NEON
#endif

Original comment by erwin.coumans on 24 Sep 2013 at 7:13

GoogleCodeExporter commented 9 years ago 
I don't get any compile error in 64bit mode, can you copy the compile error you 
get?

Thanks,
Erwin

Original comment by erwin.coumans on 6 Oct 2013 at 8:30

GoogleCodeExporter commented 9 years ago 
See also http://bulletphysics.org/Bullet/phpBB3/viewtopic.php?t=9538

I haven't been able to reproduce it yet, but we probably should disable NEON on 
64bit for now, with the pre-processor definition.
#if defined(__LP64__) || defined(_LP64)

Original comment by erwin.coumans on 11 Nov 2013 at 6:25

GoogleCodeExporter commented 9 years ago 
it should be fixed in latest trunk:
https://code.google.com/p/bullet/source/detail?r=2716

can you please test and confirm?
Thanks,
Erwin

Original comment by erwin.coumans on 18 Nov 2013 at 10:17

GoogleCodeExporter commented 9 years ago 
mark as closed, if the issue is still there please reply (or create a new issue)

Original comment by erwin.coumans on 4 Dec 2013 at 7:12