"Ronald S. Bultje" <[email protected]> writes: > Hi, > > 2011/11/11 Måns Rullgård <[email protected]>: >> "Ronald S. Bultje" <[email protected]> writes: >>> 2011/11/11 Måns Rullgård <[email protected]>: >>>> Please go and read about how altivec unaligned loads are done. >>>> If src+srcPos is always 16-byte aligned, the second line is not needed >>>> at all. If not, the overread is necessary. >>> >>> Let's just commit this padding before we bikeshed a full year about this... >> >> It's not bikeshedding. If the address is always aligned, the altivec >> code can be simplified removing the overread. Does the corresponding >> x86 code allow unaligned addresses? > > Yes. > > As said, this is for the case where we read planar YUV, then > SwsContext->convertData[] is not used, instead the planar pointer is > used directly without any conversion. This is used-provided and may > thus be unaligned (we currently make no alignment requirements).
Then apply the padding patch. -- Måns Rullgård [email protected] _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
