GoogleCodeExporter
commented
9 years ago [deleted comment]Closed GoogleCodeExporter closed 9 years ago
GoogleCodeExporter
commented
9 years ago [deleted comment]Can you make sure to define USE_ADDR64?
See Bullet\Extras\BulletMultiThreaded\PpuAddressSpace.h
This is only related to BulletMultiThreaded, which is pure optional. You should
be
able to compile the core libraries fine
(BulletCollision/BulletDynamics/LinearMath)
in 64bit mode.
Alternatively, remove the BulletMultiThreaded project, and it should be be fine.
Thanks,
ErwinOriginal comment by erwin.coumans on 7 Mar 2008 at 11:20
GoogleCodeExporter
commented
9 years ago Original comment by erwin.coumans on 8 Mar 2008 at 3:46
GoogleCodeExporter
commented
9 years ago I checked this out in svn 1106 and found that even with USE_ADDR64 there are
still
some casts from pointers to uint32_t.
Attached is a patch for consideration that introduces a slightly different
change.
I've replaced instances of ppu_address_t (as well as the uint32_t's that cause
problems) with size_t. Try it out, give it a think.Original comment by mattb...@gmail.com on 31 Mar 2008 at 2:41
Attachments:
GoogleCodeExporter
commented
9 years ago Well, that patch I posted isn't a complete fix... BulletMultiThreaded compiles
fine,
but I still have to define USE_ADDR64 to get the demos to build/link correctly.
So
please disregard or salvage it by replacing size_t with ppu_address_t
Here's what I get if you're curious:
/home/matt/src/bullet-svn-1106/Extras/BulletMultiThreaded/libLibBulletMultiThrea
ded.a(SpuGatheringCollisionTask.o):
In function `ProcessConvexConcaveSpuCollision(SpuCollisionPairInput*,
CollisionTask_LocalStoreMemory*, SpuContactResult&)':
SpuGatheringCollisionTask.cpp:(.text+0xdd6): undefined reference to
`computeAabb(btVector3&, btVector3&, btConvexInternalShape*, unsigned int, int,
btTransform)'
SpuGatheringCollisionTask.cpp:(.text+0x100f): undefined reference to
`dmaBvhSubTreeHeaders(btBvhSubtreeInfo*, unsigned int, int, unsigned int)'
/home/matt/src/bullet-svn-1106/Extras/BulletMultiThreaded/libLibBulletMultiThrea
ded.a(SpuGatheringCollisionTask.o):
In function `handleCollisionPair(SpuCollisionPairInput&,
CollisionTask_LocalStoreMemory&, SpuContactResult&, unsigned long, void*,
unsigned
long, void*, bool)':
SpuGatheringCollisionTask.cpp:(.text+0x1297): undefined reference to
`dmaCollisionShape(void*, unsigned int, unsigned int, int)'
SpuGatheringCollisionTask.cpp:(.text+0x12b3): undefined reference to
`dmaCollisionShape(void*, unsigned int, unsigned int, int)'
SpuGatheringCollisionTask.cpp:(.text+0x1feb): undefined reference to
`dmaCollisionShape(void*, unsigned int, unsigned int, int)'
SpuGatheringCollisionTask.cpp:(.text+0x2007): undefined reference to
`dmaCollisionShape(void*, unsigned int, unsigned int, int)'
SpuGatheringCollisionTask.cpp:(.text+0x20d4): undefined reference to
`dmaCollisionShape(void*, unsigned int, unsigned int, int)'
/home/matt/src/bullet-svn-1106/Extras/BulletMultiThreaded/libLibBulletMultiThrea
ded.a(SpuGatheringCollisionTask.o):SpuGatheringCollisionTask.cpp:(.text+0x291e):
more undefined references to `dmaCollisionShape(void*, unsigned int, unsigned
int,
int)' follow
Original comment by mattb...@gmail.com on 31 Mar 2008 at 3:45
GoogleCodeExporter
commented
9 years ago
Did you try to simply remove the project BulletMultiThreaded?
This project is mainly for PS3 Cell SPUs and XBox 360. Other platforms haven't
really been optimized, so it is better to use the non-parallel version:
btCollisionDispatcher, and btSequentialImpulseConstraintSolver.
We should remove the dependencies of AllBulletDemos to BulletMultiThreaded for
upcoming Bullet 2.68.
In the meanwhile, I'll see if we can make this compile, but it is low priority
(unless you really want to start optimizing this BulletMultiThreaded).
Original comment by erwin.coumans on 31 Mar 2008 at 4:18
GoogleCodeExporter
commented
9 years ago Removing BulletMultiThreaded fixes it. I understand that it's low priority, and
I
agree that the best fix is to simply remove the dependency. Just wanted to see
if I
could get it to work. :)Original comment by mattb...@gmail.com on 31 Mar 2008 at 4:52
GoogleCodeExporter
commented
9 years ago We removed it from the CMakeList.txt
Unless we receive a working/tested patch, the work be postponed for later.
Thanks,
Erwin
Original comment by erwin.coumans on 14 Sep 2008 at 6:30
Original issue reported on code.google.com by
xilo...@gmail.comon 7 Mar 2008 at 12:08