Re: [flac-dev] 1.21 vs 1.3 encoding speed
On Sat, Jul 26, 2014 at 06:29:36PM +1000, Erik de Castro Lopo wrote: Martijn van Beurden wrote: See the following lines in the configure script if test x$user_cflags = x; then CFLAGS=-O3 -funroll-loops -Wall -W -Winline fi Maybe we should change this 'overriding' behaviour of CFLAGS and add -O3 just like most other options are added to the CFLAGS? Wow, I didn't even know that was there. I have now replaced the above with: CFLAGS=-O3 -funroll-loops -Wall -W -Winline $CFLAGS so CFLAGS are preserved and enforce optimisations. Well, this reverts the commit 18e0154. How is the user supposed to set CFLAGS without getting -O3 -funroll-loops there? (e.g. to minimize the size of the compiled binaries) -- Miroslav Lichvar ___ flac-dev mailing list flac-dev@xiph.org http://lists.xiph.org/mailman/listinfo/flac-dev
Re: [flac-dev] 1.21 vs 1.3 encoding speed
Miroslav Lichvar wrote: Well, this reverts the commit 18e0154. How is the user supposed to set CFLAGS without getting -O3 -funroll-loops there? (e.g. to minimize the size of the compiled binaries) Is it -O3 or -funroll-loops that you have a problem with? Or both? What would you prefer to see as the default optimisation level? I'm sure there is a solution to this. Lets find it. Erik -- -- Erik de Castro Lopo http://www.mega-nerd.com/ ___ flac-dev mailing list flac-dev@xiph.org http://lists.xiph.org/mailman/listinfo/flac-dev
Re: [flac-dev] 1.21 vs 1.3 encoding speed
On Tue, Jul 29, 2014 at 07:47:33PM +1000, Erik de Castro Lopo wrote: Miroslav Lichvar wrote: Well, this reverts the commit 18e0154. How is the user supposed to set CFLAGS without getting -O3 -funroll-loops there? (e.g. to minimize the size of the compiled binaries) Is it -O3 or -funroll-loops that you have a problem with? Or both? What would you prefer to see as the default optimisation level? The -funroll-loops option improves the encoding speed, but it seems to increase the size significantly. I'm not suggesting to remove it from the default CFLAGS, just have an option to fully override them. I'm sure there is a solution to this. Lets find it. It would be nice if -funroll-loops was used only for the stream_encoder.c file, that's where it seems to matter most. I'm not sure if there is a better way to do this in automake, in the Fedora flac package I used this patch: http://pkgs.fedoraproject.org/cgit/flac.git/plain/flac-1.2.1-cflags.patch?id=66a59af0bdc5ae4a719aac5d4a8c41817906a01f -- Miroslav Lichvar ___ flac-dev mailing list flac-dev@xiph.org http://lists.xiph.org/mailman/listinfo/flac-dev
Re: [flac-dev] 1.21 vs 1.3 encoding speed
Miroslav Lichvar wrote: The -funroll-loops option improves the encoding speed, but it seems to increase the size significantly. I'm not suggesting to remove it from the default CFLAGS, just have an option to fully override them. It would be nice if -funroll-loops was used only for the stream_encoder.c file, that's where it seems to matter most. I'm not sure if there is a better way to do this in automake, in the Fedora flac package I used this patch: http://pkgs.fedoraproject.org/cgit/flac.git/plain/flac-1.2.1-cflags.patch?id=66a59af0bdc5ae4a719aac5d4a8c41817906a01f Ugly, but that seemse to be the standard solution to this: https://www.gnu.org/software/automake/manual/html_node/Per_002dObject-Flags.html Let me play with this. Erik -- -- Erik de Castro Lopo http://www.mega-nerd.com/ ___ flac-dev mailing list flac-dev@xiph.org http://lists.xiph.org/mailman/listinfo/flac-dev
Re: [flac-dev] 1.21 vs 1.3 encoding speed
Erik de Castro Lopo wrote: Wow, I didn't even know that was there. I have now replaced the above with: CFLAGS=-O3 -funroll-loops -Wall -W -Winline $CFLAGS so CFLAGS are preserved and enforce optimisations. I just built FLAC and noticed that the size of flac.exe is noticeably bigger, so I compared the generated Makefiles before ang after this change. The difference is: -g -O2 options were added to CFLAGS. before: CFLAGS = -O3 -funroll-loops -Wall -W -Winline -Wall -Wextra -Wstrict-prototypes -Wmissing-prototypes -Waggregate-return -Wcast-align -Wnested-externs -Wshadow -Wundef -Wmissing-declarations -Wunreachable-code -Wdeclaration-after-statement -D_FORTIFY_SOURCE=2 -msse2 -Wextra after: CFLAGS = -O3 -funroll-loops -Wall -W -Winline -g -O2 -Wall -Wextra -Wstrict-prototypes -Wmissing-prototypes -Waggregate-return -Wcast-align -Wnested-externs -Wshadow -Wundef -Wmissing-declarations -Wunreachable-code -Wdeclaration-after-statement -D_FORTIFY_SOURCE=2 -msse2 -Wextra ___ flac-dev mailing list flac-dev@xiph.org http://lists.xiph.org/mailman/listinfo/flac-dev
[flac-dev] 1.21 vs 1.3 encoding speed
Hello, I'm on a Mac and I'm noticing that encoding via the flac command line is significantly slower with version 1.3.0 than 1.2.1. I'm encoding a 24/96 file to flac, both from wav and aiff and both formats are showing the same speed decrease when using 1.3.0. To give an idea of the speed decrease, encoding at flac level 8: 24/96 wav file 1.21: 61.05 seconds. ratio=0.690 1.3: 222.48 seconds. ratio=0.690 smaller 16/44.1 wav file 1.21: 14.28 seconds. ratio=0.487 1.3: 51.21 seconds. ratio=0.487 Also, 1.2.1 will not encode an aiff-c file, but 1.3.0 will. Is this change documented anywhere? Thanks, Scott ___ flac-dev mailing list flac-dev@xiph.org http://lists.xiph.org/mailman/listinfo/flac-dev
Re: [flac-dev] 1.21 vs 1.3 encoding speed
Scott Brown scottcbr...@gmail.com писал(а) в своём письме Fri, 25 Jul 2014 21:32:50 +0400: a. Intel 2.8 Ghz Core I7 (dual core, I7-4558U) in late 2013 Macbook Pro with Retina Display b. I compiled it the same way I compiled 1.2.1: ./configure -enable-static -disable-shared CFLAGS= -isysroot /Developer/SDKs/MacOSX10.6.sdk -mmacosx-version-min=10.6 make Well... ./configure script creates config.h file. Post its contents here. ___ flac-dev mailing list flac-dev@xiph.org http://lists.xiph.org/mailman/listinfo/flac-dev