On 05/21/2018 07:55 AM, marxin wrote: > > gcc/ChangeLog: > > 2017-04-18 Denys Vlasenko <dvlas...@redhat.com> > > * config/i386/i386-common.c (ix86_handle_option): Remove support > for obsolete -malign-loops, -malign-jumps and -malign-functions > options. > * config/i386/i386.opt: Likewise. The ChangeLog doesn't seem match the actual changes. I think this patch is more about using the additional info that's potentially provided by the -falign options to drive backend decisions and dropping the secondary alignment request.
The change seems to consistently do: (1<<(LOG)))-1) Which needs some horizontal whitespace. The meat of the changes seem fairly reasonable. jeff
