"Ronald S. Bultje" <[email protected]> writes: > Hi, > > 2011/12/16 Måns Rullgård <[email protected]> > >> Måns Rullgård <[email protected]> writes: >> >> > Martin Storsjö <[email protected]> writes: >> > >> >> On Fri, 16 Dec 2011, Mans Rullgard wrote: >> >> >> >>> diff --git a/configure b/configure >> >>> index 974e75f..0349862 100755 >> >>> --- a/configure >> >>> +++ b/configure >> >>> @@ -113,6 +113,7 @@ Configuration options: >> >>> --disable-dxva2 disable DXVA2 code >> >>> --enable-runtime-cpudetect detect cpu capabilities at runtime >> (bigger binary) >> >>> --enable-hardcoded-tables use hardcoded tables instead of runtime >> generation >> >>> + --disable-safe-bitstream-reader disable buffer boundary checking in >> bitreaders (faster, but may crash) >> >>> --enable-memalign-hack emulate memalign, interferes with memory >> debuggers >> >>> --disable-everything disable all components listed below >> >>> --disable-encoder=NAME disable encoder NAME >> >>> @@ -976,6 +977,7 @@ CONFIG_LIST=" >> >>> rdft >> >>> rtpdec >> >>> runtime_cpudetect >> >>> + safe_bitstream_reader >> >>> shared >> >>> sinewin >> >>> small >> >> >> >> Doesn't this still need to be enabled somewhere, or the help text >> >> changed into --enable? >> > >> > Yes, I'm running some benchmarks now to determine whether to enable this >> > by default. We'll update the patch depending on the outcome. >> >> The performance penalty on ARM (both Cortex-A8 and A9) is 0-2% for most >> files, 4% in extreme cases. In many cases, the change is within the >> error margin of the measurement. >> >> I will not object to turning this on by default. >> > > Awesome, if you can add the proper line to configure, patch is OK with me. > Feel free to push directly.
What about changing +1 to +8? -- Måns Rullgård [email protected] _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
