[PD] soundtouch~, slicerec/sliceplay for 64 bit windows?
Christof Ressi
christof.ressi at gmx.at
Wed Oct 30 08:45:17 CET 2019
> i added the -fpermissive flag and it built :-)
Bad idea. The external will probably crash at runtime. The last error was the same problem as before, so just replace “unsigned long“ with “ulong“ (which should be unsigned long long on win64).
Actually, one clean solution is to replace “ulong“ with “uintptr_t“ (pointer sized unsigned integer) from “stdint.h“.
Just report this problem to the author and they will fix it.
> Gesendet: Mittwoch, 30. Oktober 2019 um 06:21 Uhr
> Von: "pat pagano" <shreeswifty at gmail.com>
> An: "Christof Ressi" <christof.ressi at gmx.at>
> Betreff: Re: Re: [PD] soundtouch~, slicerec/sliceplay for 64 bit windows?
>
> i got it
> i added the -fpermissive flag and it built :-)
>
>
> On Wed, Oct 30, 2019 at 12:55 AM pat pagano <shreeswifty at gmail.com> wrote:
>
> > i cleaned it up and i got it down to just one error
> >
> > C:\Users\shree\Desktop\KATJA\soundtouch_\soundtouch~\src>make
> > g++ -msse -DPD -DMSW -Werror -Wno-unused -mms-bitfields -Wno-parentheses
> > -Wno-switch -O6 -funroll-loops -fomit-frame-pointer -fno-strict-aliasing
> > -fcheck-new -I ./include -c *.cpp
> > sse_optimized.cpp: In member function 'virtual void
> > soundtouch::FIRFilterSSE::setCoefficients(const float*, uint, uint)':
> > sse_optimized.cpp:288:51: error: cast from 'float*' to 'long unsigned int'
> > loses precision [-fpermissive]
> > filterCoeffsAlign = (float *)(((unsigned long)filterCoeffsUnalign +
> > 15) & (ulong)-16);
> > ^~~~~~~~~~~~~~~~~~~
> > make: *** [makefile:69: soundtouch~.dll] Error 1
> >
> > can you help me a little more?
> >
> > best
> >
> > Patrick
> >
> >
> > On Tue, Oct 29, 2019 at 9:00 PM Christof Ressi <christof.ressi at gmx.at>
> > wrote:
> >
> >> The problem is actually in the code:
> >>
> >> (SAMPLETYPE *)(((ulong)tempUnaligned + 15) & (ulong)-16);
> >>
> >> This line probably tries to align memory on the stack to 16 bytes
> >> boundaries for SSE code. The problem is that "ulong" is a typedef for
> >> "unsigned long". The code assumes that "long" always has the size of a
> >> pointer (4 bytes on 32 bit and 8 bytes on 64 bit), but on Windows it is
> >> always 4 bytes, so on 64 bit you get a size mismatch.
> >>
> >> A quick and dirty fix: find the relevant "#define" or "typedef" for
> >> "ulong" and set it to "unsigned long long" for 64 bit Windows:
> >>
> >> #ifdef _WIN64
> >> #define ulong unsigned long long
> >> #else
> >> #define ulong unsigned long
> >> #endif
> >>
> >> Please make a bug report to the upstream author.
> >>
> >> Christof
> >> *Gesendet:* Mittwoch, 30. Oktober 2019 um 00:46 Uhr
> >> *Von:* "pat pagano" <shreeswifty at gmail.com>
> >> *An:* pd-list at iem.at
> >> *Betreff:* Re: [PD] soundtouch~, slicerec/sliceplay for 64 bit windows?
> >>
> >> I know that i am flailing with windows makefiles but i was hopeful that
> >> from trying to compile such an essential external for a few patches i've
> >> written with it I would learn compiling externals more clearly for pd and
> >> of course windows
> >>
> >> here's what i get:
> >>
> >> g++ -m64 -msse -DNT -DPD_LONGINTTYPE=__int64 -DMSW -Wall -Wextra -Wshadow
> >> -Wint-to-pointer-cast -Winline -Wstrict-aliasing -O3 -ffast-math
> >> -funroll-loops -fomit-frame-pointer -march=core2 -msse -msse2 -msse3
> >> -mfpmath=sse -I ./include -c *.cpp
> >> FIFOSampleBuffer.cpp: In member function 'void
> >> soundtouch::FIFOSampleBuffer::ensureCapacity(uint)':
> >> FIFOSampleBuffer.cpp:181:39: error: cast from 'soundtouch::SAMPLETYPE*'
> >> {aka 'float*'} to 'ulong' {aka 'long unsigned int'} loses precision
> >> [-fpermissive]
> >> temp = (SAMPLETYPE *)(((ulong)tempUnaligned + 15) & (ulong)-16);
> >> ^~~~~~~~~~~~~
> >> FIFOSampleBuffer.cpp:181:71: warning: cast to pointer from integer of
> >> different size [-Wint-to-pointer-cast]
> >> temp = (SAMPLETYPE *)(((ulong)tempUnaligned + 15) & (ulong)-16);
> >> ^
> >> FIRFilter.cpp: In static member function 'static void*
> >> soundtouch::FIRFilter::operator new(size_t)':
> >> FIRFilter.cpp:222:39: warning: unused parameter 's' [-Wunused-parameter]
> >> void * FIRFilter::operator new(size_t s)
> >> ~~~~~~~^
> >> RateTransposer.cpp: In static member function 'static void*
> >> soundtouch::RateTransposer::operator new(size_t)':
> >> RateTransposer.cpp:109:44: warning: unused parameter 's'
> >> [-Wunused-parameter]
> >> void * RateTransposer::operator new(size_t s)
> >> ~~~~~~~^
> >> soundtouch~.cpp: In function 't_int* soundtouch_perform(t_int*)':
> >> soundtouch~.cpp:91:39: warning: cast to pointer from integer of different
> >> size [-Wint-to-pointer-cast]
> >> t_soundtouch *x = (t_soundtouch *)w[1]; // pointer to object
> >> (struct instance)
> >> ^
> >> soundtouch~.cpp:92:43: warning: cast to pointer from integer of different
> >> size [-Wint-to-pointer-cast]
> >> t_sample *signalvectorIn = (t_sample*)w[2]; // pointer to input
> >> signal vector
> >> ^
> >> soundtouch~.cpp:93:44: warning: cast to pointer from integer of different
> >> size [-Wint-to-pointer-cast]
> >> t_sample *signalvectorOut = (t_sample*)w[3]; // pointer to output
> >> signal vector
> >> ^
> >> soundtouch~.cpp: In function 't_int* soundtouch_perform2(t_int*)':
> >> soundtouch~.cpp:115:39: warning: cast to pointer from integer of
> >> different size [-Wint-to-pointer-cast]
> >> t_soundtouch *x = (t_soundtouch *)w[1]; // pointer to object
> >> (struct instance)
> >> ^
> >> soundtouch~.cpp:116:47: warning: cast to pointer from integer of
> >> different size [-Wint-to-pointer-cast]
> >> t_sample *signalvectorInLeft = (t_sample*)w[2]; // pointer to
> >> input signal vector
> >> ^
> >> soundtouch~.cpp:117:48: warning: cast to pointer from integer of
> >> different size [-Wint-to-pointer-cast]
> >> t_sample *signalvectorInRight = (t_sample*)w[3]; // pointer to
> >> input signal vector
> >> ^
> >> soundtouch~.cpp:118:48: warning: cast to pointer from integer of
> >> different size [-Wint-to-pointer-cast]
> >> t_sample *signalvectorOutLeft = (t_sample*)w[4]; // pointer to
> >> output signal vector
> >> ^
> >> soundtouch~.cpp:119:49: warning: cast to pointer from integer of
> >> different size [-Wint-to-pointer-cast]
> >> t_sample *signalvectorOutRight = (t_sample*)w[5]; // pointer to
> >> output signal vector
> >> ^
> >> soundtouch~.cpp: In function 'void soundtouch_tilde_setup()':
> >> soundtouch~.cpp:312:67: warning: cast between incompatible function types
> >> from 'void* (*)(t_floatarg, t_floatarg)' {aka 'void* (*)(float, float)'} to
> >> 't_newmethod' {aka 'void* (*)()'} [-Wcast-function-type]
> >> soundtouch_class = class_new(gensym("soundtouch~"),
> >> (t_newmethod)soundtouch_new,
> >>
> >> ^~~~~~~~~~~~~~
> >> sse_optimized.cpp: In member function 'virtual double
> >> soundtouch::TDStretchSSE::calcCrossCorrStereo(const float*, const float*)
> >> const':
> >> sse_optimized.cpp:96:17: error: cast from 'const float*' to 'ulong' {aka
> >> 'long unsigned int'} loses precision [-fpermissive]
> >> if (((ulong)pV1) & 15) return -1e50; // skip unaligned locations
> >> ^~~
> >> sse_optimized.cpp: In member function 'virtual void
> >> soundtouch::FIRFilterSSE::setCoefficients(const float*, uint, uint)':
> >> sse_optimized.cpp:288:51: error: cast from 'float*' to 'long unsigned
> >> int' loses precision [-fpermissive]
> >> filterCoeffsAlign = (float *)(((unsigned long)filterCoeffsUnalign +
> >> 15) & (ulong)-16);
> >> ^~~~~~~~~~~~~~~~~~~
> >> sse_optimized.cpp:288:89: warning: cast to pointer from integer of
> >> different size [-Wint-to-pointer-cast]
> >> filterCoeffsAlign = (float *)(((unsigned long)filterCoeffsUnalign +
> >> 15) & (ulong)-16);
> >>
> >> ^
> >> In file included from ./include/FIFOSamplePipe.h:51,
> >> from RateTransposer.h:50,
> >> from TDStretch.h:49,
> >> from sse_optimized.cpp:69:
> >> sse_optimized.cpp: In member function 'virtual uint
> >> soundtouch::FIRFilterSSE::evaluateFilterStereo(float*, const float*, uint)
> >> const':
> >> sse_optimized.cpp:316:20: error: cast from 'float*' to 'ulong' {aka 'long
> >> unsigned int'} loses precision [-fpermissive]
> >> assert(((ulong)filterCoeffsAlign) % 16 == 0);
> >> ^~~~~~~~~~~~~~~~~
> >> TDStretch.cpp: In member function 'void
> >> soundtouch::TDStretch::acceptNewOverlapLength(int)':
> >> TDStretch.cpp:724:49: error: cast from 'soundtouch::SAMPLETYPE*' {aka
> >> 'float*'} to 'ulong' {aka 'long unsigned int'} loses precision
> >> [-fpermissive]
> >> pRefMidBuffer = (SAMPLETYPE *)((((ulong)pRefMidBufferUnaligned)
> >> + 15) & (ulong)-16);
> >> ^~~~~~~~~~~~~~~~~~~~~~
> >> TDStretch.cpp:724:91: warning: cast to pointer from integer of different
> >> size [-Wint-to-pointer-cast]
> >> pRefMidBuffer = (SAMPLETYPE *)((((ulong)pRefMidBufferUnaligned)
> >> + 15) & (ulong)-16);
> >>
> >> ^
> >> TDStretch.cpp: In static member function 'static void*
> >> soundtouch::TDStretch::operator new(size_t)':
> >> TDStretch.cpp:731:39: warning: unused parameter 's' [-Wunused-parameter]
> >> void * TDStretch::operator new(size_t s)
> >> ~~~~~~~^
> >>
> >> On Tue, Oct 29, 2019 at 1:51 AM IOhannes m zmölnig <zmoelnig at iem.at>
> >> wrote:
> >>
> >>> On 10/29/19 5:49 AM, pat pagano wrote:
> >>> > I tired compiling them myself but i keep getting errors so i thought i
> >>>
> >>> it seems you have forgotten to attach the errors you get.
> >>>
> >>> gfamrds
> >>> IOhannes
> >>>
> >>> _______________________________________________
> >>> Pd-list at lists.iem.at mailing list
> >>> UNSUBSCRIBE and account-management ->
> >>> https://lists.puredata.info/listinfo/pd-list
> >>
> >>
> >>
> >> --
> >> *Patrick Pagano B.S, M.F.A*
> >> *Assistant Professor, *Interactive Media & Education
> >> Audio Projection & Performance Design
> >> Howard Community College
> >> *Columbia, Maryland USA*
> >> *http://shreeswifty.github.io <http://shreeswifty.github.io>*
> >> *(352)226-2016*
> >>
> >>
> >> _______________________________________________ Pd-list at lists.iem.at
> >> mailing list UNSUBSCRIBE and account-management ->
> >> https://lists.puredata.info/listinfo/pd-list
> >>
> >
> >
> > --
> > *Patrick Pagano B.S, M.F.A*
> > *Assistant Professor, *Interactive Media & Education
> > Audio Projection & Performance Design
> > Howard Community College
> > *Columbia, Maryland USA*
> > *http://shreeswifty.github.io <http://shreeswifty.github.io>*
> > *(352)226-2016*
> >
> >
> >
>
> --
> *Patrick Pagano B.S, M.F.A*
> *Assistant Professor, *Interactive Media & Education
> Audio Projection & Performance Design
> Howard Community College
> *Columbia, Maryland USA*
> *http://shreeswifty.github.io <http://shreeswifty.github.io>*
> *(352)226-2016*
>
More information about the Pd-list
mailing list