It's safe to assume that LOCAL_ALIGNED_16 does indeed align. Otherwise we would have many more problems... This assert was added in f8188626 all the way back in 2003.
Unified · Split
| @@ -99,8 +99,6 @@ static int RENAME(dct_quantize)(MpegEncContext *s, · 98 unmodified lines | |||
| 99 | const uint16_t *qmat, *bias; | 99 | const uint16_t *qmat, *bias; |
| 100 | LOCAL_ALIGNED_16(int16_t, temp_block, [64]); | 100 | LOCAL_ALIGNED_16(int16_t, temp_block, [64]); |
| 101 | 101 | ||
| 102 | av_assert2((7&(uintptr_t)(&temp_block[0])) == 0); //did gcc align it correctly? | ||
| 103 | |||
| 104 | //s->fdct (block); | 102 | //s->fdct (block); |
| 105 | RENAME_FDCT(ff_fdct)(block); // cannot be anything else ... | 103 | RENAME_FDCT(ff_fdct)(block); // cannot be anything else ... |
| 106 | 104 | ||