Re: [PATCH][AArch64][6/14] Implement TARGET_OPTION_SAVE/TARGET_OPTION_RESTORE

2015-07-21 Thread Kyrill Tkachov
On 21/07/15 12:09, James Greenhalgh wrote: On Fri, Jul 17, 2015 at 03:19:20PM +0100, Kyrill Tkachov wrote: This is a slight respin of this patch, handling the -moverride string more gracefully. We need to explicitly save and restore it in TARGET_OPTION_SAVE otherwise the option gen machinery

Re: [PATCH][AArch64][6/14] Implement TARGET_OPTION_SAVE/TARGET_OPTION_RESTORE

2015-07-21 Thread James Greenhalgh
On Fri, Jul 17, 2015 at 03:19:20PM +0100, Kyrill Tkachov wrote: > > > This is a slight respin of this patch, handling the -moverride string more > gracefully. > We need to explicitly save and restore it in TARGET_OPTION_SAVE otherwise the > option gen machinery > gets confused about its type an

Re: [PATCH][AArch64][6/14] Implement TARGET_OPTION_SAVE/TARGET_OPTION_RESTORE

2015-07-17 Thread Kyrill Tkachov
This is a slight respin of this patch, handling the -moverride string more gracefully. We need to explicitly save and restore it in TARGET_OPTION_SAVE otherwise the option gen machinery gets confused about its type and during its printing uses the wrong format code for the pointer, leading to