Ticket #5997 (closed defect: fixed)
Opened 4 years ago
Last modified 4 years ago
soundtouch SSE ASM exposes GCC bug
|Reported by:||nigel||Owned by:||ijr|
Mac gcc4.0.1 r19283 build does:
g++ -c -pipe -march=pentium4 -g -Wall -Wno-switch -Wpointer-arith -Wredundant-decls -Wno-non-virtual-dtor -funit-at-a-time -D__STDC_CONSTANT_MACROS -Wno-long-double -fPIC -DPIC -w -fPIC -DMMX -Di386 -DUSING_APPLEREMOTE -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -DALLOW_SSE2 -I/Volumes/MythBuild/.osx-packager/build/mkspecs/macx-g++ -I. -I/Volumes/MythBuild/.osx-packager/build/include -I/Volumes/MythBuild/.osx-packager/build/include -I../../libs/libavcodec -I../.. -I. -o sse_gcc.o sse_gcc.cpp sse_gcc.cpp: In member function 'virtual void soundtouch::TDStretchSSE2::overlapMulti(short int*, const short int*) const': sse_gcc.cpp:164: error: can't find a register in class 'GENERAL_REGS' while reloading 'asm'
We work around the bug in libavcodec by tweaking the compile flags, but since this ASM was recently developed for #5890, I'm wondering if we can rework it for less registers?
comment:1 Changed 4 years ago by foobum@…
comment:2 Changed 4 years ago by nigel
comment:3 Changed 4 years ago by foobum@…
comment:4 Changed 4 years ago by nigel
comment:5 Changed 4 years ago by nigel
- Status changed from new to closed
- Resolution set to fixed