I just rebased the GIL change atop the latest fixes to the SIMD code and 
re-pushed it.

        -- lg


On Sep 12, 2014, at 3:39 AM, Dave Lajoie <[email protected]> 
wrote:

> Thanks Larry, Much appreciated.
> 
> Is the following pull been updated as well?
> 
> https://github.com/OpenImageIO/oiio/pull/950 ( python GIL threading 
> improvement )
> 
> That is the one we are testing now.
> Dave.
> 
> 
> 
> Dave Lajoie
> R&D Director |  Directeur R&D
>  
> 5605 Avenue de Gaspé, Suite 408  |  Montréal, QC H2T 2A4 
> Tel  (514) 360-3253  ext. 130
> 
> On Fri, Sep 12, 2014 at 2:29 AM, Larry Gritz <[email protected]> wrote:
> Fix committed to master. Sorry for the inconvenience.
> 
>       -- lg
> 
> 
> On Sep 11, 2014, at 9:17 PM, Etienne Fleurant <[email protected]> 
> wrote:
> 
>> Thanks a lot! Really appreciate the help!
>> 
>> Cheers!
>> De : Larry Gritz
>> Envoyé : ‎2014-‎09-‎11 22:06
>> À : OpenImageIO developers; Etienne Fleurant
>> Objet : Re: [Oiio-dev] Issue building OIIO 1.5, SIMD related error
>> 
>> I have a fix! I'm tied up at the moment, but will submit the fix later 
>> tonight. 
>> 
>> 
>> 
>> On September 11, 2014 12:26:24 PM PDT, Etienne Fleurant 
>> <[email protected]> wrote:
>> Super! Thanks a bunch!
>> 
>> 2014-09-11 15:25 GMT-04:00 Larry Gritz <[email protected]>:
>> I'll try to get this resolved today. Stay tuned.
>> 
>> We knew this change would probably break things, but we'll just git it fixed 
>> as fast as we can.
>> 
>>  -- lg
>> 
>> 
>> On Sep 11, 2014, at 12:21 PM, Etienne Fleurant <[email protected]> 
>> wrote:
>> 
>>> Here's my platform detail:
>>> 
>>> CentOS 6.5
>>> CMake 2.8.8
>>> gcc (GCC) 4.4.7 20120313 (Red Hat 4.4.7-4)
>>> Intel i7-3930K
>>> 
>>> Thanks for the lightning fast response!
>>> Cheers
>>> 
>>> 2014-09-11 15:15 GMT-04:00 Larry Gritz <[email protected]>:
>>> Platform details?
>>> 
>>> 
>>> On Sep 11, 2014, at 12:14 PM, Richard Shaw <[email protected]> wrote:
>>> 
>>>> I'm getting something similar but doesn't look exactly the same:
>>>> 
>>>> [ 10%] Building CXX object 
>>>> src/libutil/CMakeFiles/simd_test.dir/simd_test.cpp.o
>>>> cd /home/build/tmp/build_oiio/src/libutil && /usr/lib64/ccache/c++   
>>>> -DUSE_FIELD3D=1 -DUSE_FREETYPE -DUSE_OCIO=1 -DUSE_OPENSSL=1 
>>>> -D__STDC_LIMIT_MACROS -O3 -DNDEBUG -I/usr/include/OpenEXR 
>>>> -I/usr/include/libraw -I/home/build/git/oiio/src/include 
>>>> -I/home/build/tmp/build_oiio/include/OpenImageIO    -Wall -Werror 
>>>> -fno-math-errno -Wno-error=unused-local-typedefs -Wunused-local-typedefs 
>>>> -o CMakeFiles/simd_test.dir/simd_test.cpp.o -c 
>>>> /home/build/git/oiio/src/libutil/simd_test.cpp
>>>> In file included from /home/build/git/oiio/src/libutil/simd_test.cpp:32:0:
>>>> /home/build/git/oiio/src/include/OpenImageIO/simd.h: In function 
>>>> ‘OpenImageIO::v1_5::simd::mask4 
>>>> OpenImageIO::v1_5::simd::operator|(OpenImageIO::v1_5::simd::mask4, 
>>>> OpenImageIO::v1_5::simd::mask4)’:
>>>> /home/build/git/oiio/src/include/OpenImageIO/simd.h:333:46: error: cannot 
>>>> convert ‘OpenImageIO::v1_5::simd::mask4::simd_t {aka __vector(4) float}’ 
>>>> to ‘__m128i {aka __vector(2) long long int}’ for argument ‘1’ to ‘__m128i 
>>>> _mm_or_si128(__m128i, __m128i)’
>>>>          return _mm_or_si128 (a.m_vec, b.m_vec);
>>>>                                               ^
>>>> /home/build/git/oiio/src/include/OpenImageIO/simd.h: In function ‘const 
>>>> OpenImageIO::v1_5::simd::mask4 
>>>> OpenImageIO::v1_5::simd::operator==(OpenImageIO::v1_5::simd::mask4, 
>>>> OpenImageIO::v1_5::simd::mask4)’:
>>>> /home/build/git/oiio/src/include/OpenImageIO/simd.h:348:55: error: cannot 
>>>> convert ‘OpenImageIO::v1_5::simd::mask4’ to ‘__m128i {aka __vector(2) long 
>>>> long int}’ for argument ‘1’ to ‘__m128i _mm_cmpeq_epi32(__m128i, __m128i)’
>>>>          return _mm_castsi128_ps (_mm_cmpeq_epi32 (a, b));
>>>>                                                        ^
>>>> /home/build/git/oiio/src/include/OpenImageIO/simd.h:355:5: error: control 
>>>> reaches end of non-void function [-Werror=return-type]
>>>>      }
>>>>      ^
>>>> 
>>>> Thanks,
>>>> Richard
>>> 
>>> --
>>> Larry Gritz
>>> [email protected]
>>> 
>>> 
>>> 
>>> 
>>> _______________________________________________
>>> Oiio-dev mailing list
>>> [email protected]
>>> http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org
>>> 
>>> 
>>> _______________________________________________
>>> Oiio-dev mailing list
>>> [email protected]
>>> http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org
>> 
>> --
>> Larry Gritz
>> [email protected]
>> 
>> 
>> 
>> 
>> _______________________________________________
>> Oiio-dev mailing list
>> [email protected]
>> http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org
>> 
>> 
>> 
>> 
>> Oiio-dev mailing list
>> [email protected]
>> http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org
>> 
>> -- 
>> Larry Gritz
>> [email protected] 
> 
> --
> Larry Gritz
> [email protected]
> 
> 
> 
> 
> _______________________________________________
> Oiio-dev mailing list
> [email protected]
> http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org
> 
> 
> _______________________________________________
> Oiio-dev mailing list
> [email protected]
> http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org

--
Larry Gritz
[email protected]



_______________________________________________
Oiio-dev mailing list
[email protected]
http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org

Reply via email to