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

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.

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

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.

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

2016-02-17 Thread Daniel Gutson
, NULL); >> + bitfield_stmt_bfcopy_pair *p_st_cpy; >> + unsigned HOST_WIDE_INT mrg_off; >> + p_st_cpy = bf_stmnt_cpy->find (_cpy); >> + if (p_st_cpy) >> + cc = p_st_cpy->copy; >> + >> + if (cc

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

2014-11-26 Thread Andrew Pinski
: Thursday, October 16, 2014 4:09 AM To: Zoran Jovanovic Cc: gcc-patches@gcc.gnu.org; Richard Biener; Bernhard Reutner-Fischer Subject: Re: [PATCH] Add a new option -fmerge-bitfields (patch / doc inside) On Tue, Apr 22, 2014 at 3:28 AM, Zoran Jovanovic zoran.jovano...@imgtec.com wrote

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

2014-10-29 Thread Zoran Jovanovic
bool using_eh_for_cleanups_p (void); Regards, Zoran From: Andrew Pinski [pins...@gmail.com] Sent: Thursday, October 16, 2014 4:09 AM To: Zoran Jovanovic Cc: gcc-patches@gcc.gnu.org; Richard Biener; Bernhard Reutner-Fischer Subject: Re: [PATCH] Add a new option -fmerge-bitfields (patch / doc

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 zoran.jovano...@imgtec.com 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.

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 zoran.jovano...@imgtec.com 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

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-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 zoran.jovano...@imgtec.com wrote: Hello, This is new patch version. Lowering is applied only for bit-fields copy sequences that are merged. Data structure representing bit-field

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

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 zoran.jovano...@imgtec.com 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,

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 zoran.jovano...@imgtec.com 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