On Mon, Dec 19, 2016 at 10:47:09PM +0100, FX wrote: > For ABI compatibility, we kept some unused elements in the array argument to > _gfortran_set_options (options that we have removed). With the current ABI > breakage, we might as well remove those. > > Bootstrapped and regtested on x86_64-apple-darwin16.3.0 > OK to commit? > > FX >
The patch looks fine to me. Thinking out loud here. I wonder, however, if we want to future proof the library against changes to the options passed by having a few spare unused entried available. This of course only helps if a new option needs to be added. It does nothing for removal. -- Steve
