[Bug 1364618] Review Request: embree - Collection of high-performance ray tracing kernels developed at Intel

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1364618



--- Comment #8 from Igor Gnatenko <ignatenko@xxxxxxxxxx> ---
(In reply to Zbigniew Jędrzejewski-Szmek from comment #7)
> About the compiler flags: the guidelines say that distribution compilation
> flags should be used during compilation. How to achieve that is not fully
> specified, since it depends on the build system and usually there's a few
> different way to achieve the desired result.
> 
> In this case the distro flags _are_ used:
> 
> $ rpmbuild --eval %optflags
> -O2 -g -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2
> -fexceptions -fstack-protector-strong --param=ssp-buffer-size=4
> -grecord-gcc-switches -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -m64
> -mtune=generic
> 
> from build.log:
> /usr/bin/c++   -DTASKING_TBB -D__TARGET_AVX2__ -D__TARGET_AVX__
> -I/builddir/build/BUILD/embree-2.10.0/include
> -I/builddir/build/BUILD/embree-2.10.0/x86_64-redhat-linux-gnu  -O2 -g -pipe
> -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -fexceptions
> -fstack-protector-strong --param=ssp-buffer-size=4 -grecord-gcc-switches
> -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -m64 -mtune=generic  -fPIC
> -std=c++11 -fno-strict-aliasing -Wno-narrowing -fvisibility-inlines-hidden
> -fvisibility=hidden -DNDEBUG -O3   -o CMakeFiles/sys.dir/alloc.cpp.o -c
> /builddir/build/BUILD/embree-2.10.0/common/sys/alloc.cpp
> 
> So the only issue is that this package requires AVX extensions, which means
> that it will not run on processors older then ~2011. Our guidelines specify
> that earlier CPUs should be supported, but in this case I'd say that this is
> OK: this program makes use and requires this hardware capability. Compiling
> a crippled version that will run slowly on all hardware isn't useful.
actually minimal req of this library is SSE2, but optionally (by default) AVX
and AVX2 is enabled.

So it's possible without pain to disable those. Though I'm not sure about how
slower it will be without AVX*.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://lists.fedoraproject.org/admin/lists/package-review@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]