On Sun, Jul 29, 2018 at 6:27 PM Jozef Lawrynowicz <joze...@mittosystems.com> wrote: > > pr45678-2.c ICEs for msp430-elf with -mlarge, because an alignment of > POINTER_SIZE is attempted. POINTER_SIZE with -mlarge is 20-bits, so further > code in the middle-end that expects this to be a power or 2 causes odd > alignments to be set, in this case eventually resulting in an ICE. > > The test ICEs on gcc-7-branch, gcc-8-branch, and current trunk. It > successfully builds on gcc-6-branch. > The failure is caused by r235172. > > Successfully bootstrapped and regtested the attached patch for > x86-64-pc-linux-gnu, and msp430-elf with -mlarge, on trunk. > > Ok for gcc-7-branch, gcc-8-branch and trunk?
I wonder if most (if not all) places you touch want to use get_mode_alignment (Pmode) instead? (or ptr_mode) Anyhow, the patch is otherwise obvious though factoring the thing might be nice (thus my suggestion above...) Richard.