It compiles!!!! :)

A couple of things I noticed though:

1- I still have the same error than before when I leave USE_SIMD as-is...
but I can now bypass it by setting it to 0, so it's not blocking me; I was
just wondering why I can't use SSE instructions with my current setup.
Maybe my compiler version?!

2- I had a couple of warning of the following sort:
'''
[ 23%] Building CXX object
src/libOpenImageIO/CMakeFiles/OpenImageIO.dir/imagebufalgo_copy.cpp.o
cc1plus: warnings being treated as errors
/root/oiio-master/src/libOpenImageIO/imagebufalgo_copy.cpp: In function
‘bool
OpenImageIO::v1_5::ImageBufAlgo::reorient(OpenImageIO::v1_5::ImageBuf&,
const OpenImageIO::v1_5::ImageBuf&, int)’:
/root/oiio-master/src/libOpenImageIO/imagebufalgo_copy.cpp:459: error: ‘ok’
may be used uninitialized in this function
make[2]: ***
[src/libOpenImageIO/CMakeFiles/OpenImageIO.dir/imagebufalgo_copy.cpp.o]
Error 1
make[1]: *** [src/libOpenImageIO/CMakeFiles/OpenImageIO.dir/all] Error 2
make: *** [all] Error 2
'''
and
'''
[ 34%] Building CXX object
src/libOpenImageIO/CMakeFiles/OpenImageIO.dir/__/libtexture/texturesys.cpp.o
cc1plus: warnings being treated as errors
/root/oiio-master/src/libtexture/texturesys.cpp: In member function ‘bool
OpenImageIO::v1_5::pvt::TextureSystemImpl::accum_sample_bicubic(float,
float, int, OpenImageIO::v1_5::pvt::TextureSystemImpl::TextureFile&,
OpenImageIO::v1_5::pvt::TextureSystemImpl::PerThreadInfo*,
OpenImageIO::v1_5::TextureOpt&, int, int, float, float*, float*, float*)’:
/root/oiio-master/src/libtexture/texturesys.cpp:1925: error: ‘dwx[3]’ may
be used uninitialized in this function
/root/oiio-master/src/libtexture/texturesys.cpp:1925: error: ‘dwx[2]’ may
be used uninitialized in this function
/root/oiio-master/src/libtexture/texturesys.cpp:1925: error: ‘dwx[1]’ may
be used uninitialized in this function
/root/oiio-master/src/libtexture/texturesys.cpp:1925: error: ‘dwx[0]’ may
be used uninitialized in this function
/root/oiio-master/src/libtexture/texturesys.cpp:1926: error: ‘dwy[3]’ may
be used uninitialized in this function
/root/oiio-master/src/libtexture/texturesys.cpp:1926: error: ‘dwy[2]’ may
be used uninitialized in this function
/root/oiio-master/src/libtexture/texturesys.cpp:1926: error: ‘dwy[1]’ may
be used uninitialized in this function
/root/oiio-master/src/libtexture/texturesys.cpp:1926: error: ‘dwy[0]’ may
be used uninitialized in this function
make[2]: ***
[src/libOpenImageIO/CMakeFiles/OpenImageIO.dir/__/libtexture/texturesys.cpp.o]
Error 1
make[1]: *** [src/libOpenImageIO/CMakeFiles/OpenImageIO.dir/all] Error 2
make: *** [all] Error 2
'''
I am guessing it's because some variable have not been assigned?!
Anyway, it was easy to bypass them by setting STOP_ON_WARNING to false.

3- Building tiffoutput.cpp was failing until I changed uint64 dir_offset to
uint64_t dir_offset (line 574). Bad variable type :)

Apart from that, worked like a charm.

Cheers

2014-09-12 12:11 GMT-04:00 Larry Gritz <[email protected]>:

> On Sep 12, 2014, at 9:03 AM, Richard Shaw <[email protected]> wrote:
>
> >
> > Is there an automated way to apply this through git? Or should I just
> apply it to my clone of git master and just reset it later?
> >
>
> This should work, if your master is up to date:
>
> git checkout master
> git checkout -b test
> git pull https://github.com/lgritz/oiio lg-simdfix
>
>
> --
> 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

Reply via email to