"Ronald S. Bultje" <[email protected]> writes: > On Wed, Jun 29, 2011 at 3:51 PM, Jason Garrett-Glaser <[email protected]> wrote: >> --- >> libavcodec/h264.h | 247 >> +++++++++++++++++++++++++++-------------------------- >> 1 files changed, 124 insertions(+), 123 deletions(-) > > The commit msg here could be adapted a little to say that you prevent > gcc from reloading h->xx repeatedly (which is what this patch does, > afaics). > > Patch itself is OK. I wonder why gcc does this (reloading h->xx) for > you, I don't think it does for me... New "optimization" in a later > version of gcc?
Possibly better aliasing tracking. Potential aliasing is usually the reason for reloading struct fields. -- Måns Rullgård [email protected] _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
