Re: [PATCH] Add a new option "-fmerge-bitfields" (patch / doc inside)

2016-02-17 Thread Richard Biener
On Wed, 17 Feb 2016, Richard Biener wrote: > On Wed, 17 Feb 2016, Bernd Schmidt wrote: > > > > > > > On 02/17/2016 02:18 PM, Daniel Gutson wrote: > > > On Wed, Nov 26, 2014 at 5:46 AM, Andrew Pinski wrote: > > > > > > FYI. This causes gfc_add_interface_mapping in fortrant/trans-expr.c to > >

Re: [PATCH] Add a new option "-fmerge-bitfields" (patch / doc inside)

2016-02-17 Thread Richard Biener
On Wed, 17 Feb 2016, Bernd Schmidt wrote: > > > On 02/17/2016 02:18 PM, Daniel Gutson wrote: > > On Wed, Nov 26, 2014 at 5:46 AM, Andrew Pinski wrote: > > > > FYI. This causes gfc_add_interface_mapping in fortrant/trans-expr.c to > > > be miscompiled for aarch64-linux-gnu. I am still debuggin

Re: [PATCH] Add a new option "-fmerge-bitfields" (patch / doc inside)

2016-02-17 Thread Jakub Jelinek
On Wed, Feb 17, 2016 at 02:45:16PM +0100, Bernd Schmidt wrote: > > > On 02/17/2016 02:18 PM, Daniel Gutson wrote: > >On Wed, Nov 26, 2014 at 5:46 AM, Andrew Pinski wrote: > > >>FYI. This causes gfc_add_interface_mapping in fortrant/trans-expr.c to > >>be miscompiled for aarch64-linux-gnu. I am

Re: [PATCH] Add a new option "-fmerge-bitfields" (patch / doc inside)

2016-02-17 Thread Bernd Schmidt
On 02/17/2016 02:18 PM, Daniel Gutson wrote: On Wed, Nov 26, 2014 at 5:46 AM, Andrew Pinski wrote: FYI. This causes gfc_add_interface_mapping in fortrant/trans-expr.c to be miscompiled for aarch64-linux-gnu. I am still debugging it and trying to get a smaller testcase. Hello, is the

Re: [PATCH] Add a new option "-fmerge-bitfields" (patch / doc inside)

2016-02-17 Thread Daniel Gutson
f (bf_access_candidate_p (ref, &off)) >> + { >> + bfcopy *cc = NULL; >> + bitfield_stmt_bfcopy_pair st_cpy (stmt, NULL); >> + bitfield_stmt_bfcopy_pair *p_st_cpy; >> + unsigned HOST_WIDE_INT mrg_off; >> +

Re: [PATCH] Add a new option "-fmerge-bitfields" (patch / doc inside)

2014-11-26 Thread Andrew Pinski
id (void) > return next_value_id++; > } > > - > -/* Compare two expressions E1 and E2 and return true if they are equal. */ > - > -bool > -expressions_equal_p (tree e1, tree e2) > -{ > - /* The obvious case. */ > - if (e1 == e2) > -return true; > -

RE: [PATCH] Add a new option "-fmerge-bitfields" (patch / doc inside)

2014-10-29 Thread Zoran Jovanovic
P but it can be used multiple times @@ -4184,6 +4185,11 @@ inlined_function_outer_scope_p (const_tree block) (TREE = function_args_iter_cond (&(ITER))) != NULL_TREE; \ function_args_iter_next (&(ITER))) + +/* In dwarf2out.c. */ +HOST_WIDE_INT +f

Re: [PATCH] Add a new option "-fmerge-bitfields" (patch / doc inside)

2014-10-15 Thread Andrew Pinski
On Tue, Apr 22, 2014 at 3:28 AM, Zoran Jovanovic wrote: > Hello, > Updated doc/invoke.texi by stating that new option is enabled by default at > -O2 and higher. > Also, -fmerge-bitfields added to the list of optimization flags enabled by > default at -O2 and higher. With this patch applied g

RE: [PATCH] Add a new option "-fmerge-bitfields" (patch / doc inside)

2014-04-22 Thread Zoran Jovanovic
Hello, Updated doc/invoke.texi by stating that new option is enabled by default at -O2 and higher. Also, -fmerge-bitfields added to the list of optimization flags enabled by default at -O2 and higher. Regards, Zoran Jovanovic --

Re: [PATCH] Add a new option "-fmerge-bitfields" (patch / doc inside)

2014-04-19 Thread Ryan Hill
On Thu, 17 Apr 2014 11:59:16 + Zoran Jovanovic wrote: > --- a/gcc/doc/invoke.texi > +++ b/gcc/doc/invoke.texi > @@ -7789,6 +7789,11 @@ pointer alignment information. > This pass only operates on local scalar variables and is enabled by default > at @option{-O} and higher. It requires that

RE: [PATCH] Add a new option "-fmerge-bitfields" (patch / doc inside)

2014-04-17 Thread Zoran Jovanovic
: Re: [PATCH] Add a new option "-fmerge-bitfields" (patch / doc inside) On Wed, Apr 16, 2014 at 8:38 AM, Zoran Jovanovic wrote: > Hello, > This is new patch version. > Lowering is applied only for bit-fields copy sequences that are merged. > Data structure representing bit-f

RE: [PATCH] Add a new option "-fmerge-bitfields" (patch / doc inside)

2014-04-17 Thread Zoran Jovanovic
Hello, My apologies for inconvenience. Removed every appearance of -ftree-bitfield-merge from the patch and fixed an issue with unions. The rest of the patch is the same as before. Regards, Zoran Jovanovic --

Re: [PATCH] Add a new option "-fmerge-bitfields" (patch / doc inside)

2014-04-16 Thread Daniel Gutson
On Wed, Apr 16, 2014 at 8:38 AM, Zoran Jovanovic wrote: > Hello, > This is new patch version. > Lowering is applied only for bit-fields copy sequences that are merged. > Data structure representing bit-field copy sequences is renamed and reduced > in size. > Optimization turned on by default for

Re: [PATCH] Add a new option "-fmerge-bitfields" (patch / doc inside)

2014-04-16 Thread Bernhard Reutner-Fischer
On 16 April 2014 13:38, Zoran Jovanovic wrote: > Hello, > This is new patch version. The comment from the previous iteration still holds true: > +@item -fbitfield-merge you are talking about '-fmerge-bitfields' up until here. Please fix all occurances of "bitfield-merge", both in the docs as we

RE: [PATCH] Add a new option "-fmerge-bitfields" (patch / doc inside)

2014-04-16 Thread Zoran Jovanovic
Hello, This is new patch version. Lowering is applied only for bit-fields copy sequences that are merged. Data structure representing bit-field copy sequences is renamed and reduced in size. Optimization turned on by default for -O2 and higher. Some comments fixed. Benchmarking performed on WebK