On Thu, Jun 12, 2014 at 6:33 AM, Jan Hubicka wrote:
> Hi,
> this lenghtly patch makes the legwork to put section names out of tree
> representation.
> Originally they were STRING_CST. I ended up implementing on-side reference
> counted
> string voclabulary that is done in bit baroque way to be G
On Wed, Jun 11, 2014 at 6:08 PM, Ilya Enkovich wrote:
> Hi,
>
> This patch fixes problem with size emitted for static structures with
> flexible array. I found a couple of trackers in guzilla for this problem but
> all of them are marked as fixed and problem still exists.
>
> For a simple testc
On Wed, Jun 11, 2014 at 11:35 PM, DJ Delorie wrote:
>
> If the combined bitfields are exactly the size of the mode, the logic
> for detecting range overflow is flawed - it calculates an ending
> "position" that's the position of the first bit in the next field.
>
> In the case of "short" for examp
> This patch tries to get safe lower and upper bounds where accesses
> are always guaranteed to work. The goal is not to penalize
> reasonable written code: When boot-strapping the whole GCC
> only a few places were found, where this new check triggers.
>
> Boot-strapped and regression-tested on
On Thu, Jun 12, 2014 at 10:03 AM, Eric Botcazou wrote:
>> This patch tries to get safe lower and upper bounds where accesses
>> are always guaranteed to work. The goal is not to penalize
>> reasonable written code: When boot-strapping the whole GCC
>> only a few places were found, where this new
> On Thu, Jun 12, 2014 at 6:33 AM, Jan Hubicka wrote:
> > Hi,
> > this lenghtly patch makes the legwork to put section names out of tree
> > representation.
> > Originally they were STRING_CST. I ended up implementing on-side reference
> > counted
> > string voclabulary that is done in bit baroq
On Thu, Jun 12, 2014 at 6:04 AM, Evgeny Stupachenko wrote:
> Testing finished. No new regressions.
> Is the following patch ok?
+ if (targetm.sched.reassociation_width (VEC_PERM_EXPR, mode) > 1 ||
+ !vect_shift_permute_load_chain (dr_chain, size, stmt, gsi,
&result_chain))
||s and &&s go t
> If we want to give frontends a way to pass information that address of a
> given global object is not taken (apparently useful for Ada and its alias
> attribute), then I do not think we are looking for middle-end only
> solution.
I don't feel very confortable with doing that in Ada, since everyb
On Thu, 12 Jun 2014, Eric Botcazou wrote:
> > If we want to give frontends a way to pass information that address of a
> > given global object is not taken (apparently useful for Ada and its alias
> > attribute), then I do not think we are looking for middle-end only
> > solution.
>
> I don't fee
> Btw, I wonder if we can simply mark the MEMs generated from spill code
> with MEM_NOTRAP_P so we can remove the special casing of
> frame-pointer-based addresses from add while properly initializing
> MEM_NOTRAP_p from rtx_addr_can_trap_p?
Spill code generated by the compiler itself? That's qui
On Thu, 12 Jun 2014 10:36:25, Eric Botcazou wrote:
>
>> Btw, I wonder if we can simply mark the MEMs generated from spill code
>> with MEM_NOTRAP_P so we can remove the special casing of
>> frame-pointer-based addresses from add while properly initializing
>> MEM_NOTRAP_p from rtx_addr_can_trap_p?
Richard,
as briefly discussed before, I would like to teach LTO type merging to not merge
types that was declared in anonymous namespaces and use C++ ODR type names
(stored in DECL_ASSEMBLER_NAME of the TYPE_DECL) to break down canonical types
by their names.
First thing I need to arrange IMO is
> On Thu, 12 Jun 2014, Eric Botcazou wrote:
>
> > > If we want to give frontends a way to pass information that address of a
> > > given global object is not taken (apparently useful for Ada and its alias
> > > attribute), then I do not think we are looking for middle-end only
> > > solution.
> >
> Btw I am not sure at all, why argp-references can never be dangerous?
> For instance in a struct with an array inside, passed as function argument?
IMO there cannot be any definitive solution to this issue until after we move
all the affected optimizations from RTL to GIMPLE. In the meantime,
> Btw, may_be_aliased already does that.
Indeed, and we could make use of that in Ada, at least in some cases.
> Of course one issue is that it's impossible to write a verifier that
> checks whether DECL_NONALIASED and TREE_ADDRESSABLE are "out-of-sync"
> (because by design they can be). So it's
On Thu, 12 Jun 2014, Jan Hubicka wrote:
> > On Thu, 12 Jun 2014, Eric Botcazou wrote:
> >
> > > > If we want to give frontends a way to pass information that address of a
> > > > given global object is not taken (apparently useful for Ada and its
> > > > alias
> > > > attribute), then I do not t
There was apparently a last-minute name change for DECL_NONALIASED.
Tested on x86_64-suse-linux, applied on mainline and 4.9 branch as obvious.
2014-06-12 Eric Botcazou
* tree-core.h (DECL_NONALIASED): Use proper spelling in comment.
--
Eric BotcazouIndex: tree-core.h
On Thu, 12 Jun 2014 10:50:29, Eric Botcazou wrote:
>
>> Btw I am not sure at all, why argp-references can never be dangerous?
>> For instance in a struct with an array inside, passed as function argument?
>
> IMO there cannot be any definitive solution to this issue until after we move
> all the af
This implements the requested inlining of memmove for possibly
overlapping arguments by doing first all loads and then all stores.
The easiest place is to do this in memory op folding where we already
perform inlining of some memcpy cases (but fail to do the equivalent
memcpy optimization - though
Hi DJ,
> Have you checked the other alignment macros to see if they need to be
> fixed too?
Thank you for review this patch.
Yes, I have checked other alignment macros and it seems fine.
> This should be :
I have corrected this review comment.
Is this patch now ok to commit?
Best Regards,
Sand
On Thu, Jun 12, 2014 at 10:47 AM, Jan Hubicka wrote:
> Richard,
> as briefly discussed before, I would like to teach LTO type merging to not
> merge
> types that was declared in anonymous namespaces and use C++ ODR type names
> (stored in DECL_ASSEMBLER_NAME of the TYPE_DECL) to break down canoni
On Thu, Jun 12, 2014 at 12:29 PM, Richard Biener
wrote:
> On Thu, Jun 12, 2014 at 10:47 AM, Jan Hubicka wrote:
>> Richard,
>> as briefly discussed before, I would like to teach LTO type merging to not
>> merge
>> types that was declared in anonymous namespaces and use C++ ODR type names
>> (stor
Hi Honza,
>> Unfortunately, AIX isn't the only target massively affected by your
>> recent patches. This all started with r210597
>>
>> 2014-05-17 Jan Hubicka
>>
>> * tree-pass.h (make_pass_ipa_comdats): New pass.
>> * timevar.def (TV_IPA_COMDATS): New timevar.
>> * pass
On Thu, Jun 12, 2014 at 12:34 PM, Richard Biener
wrote:
> On Thu, Jun 12, 2014 at 12:29 PM, Richard Biener
> wrote:
>> On Thu, Jun 12, 2014 at 10:47 AM, Jan Hubicka wrote:
>>> Richard,
>>> as briefly discussed before, I would like to teach LTO type merging to not
>>> merge
>>> types that was de
On Thu, Jun 12, 2014 at 10:36 AM, Eric Botcazou wrote:
>> Btw, I wonder if we can simply mark the MEMs generated from spill code
>> with MEM_NOTRAP_P so we can remove the special casing of
>> frame-pointer-based addresses from add while properly initializing
>> MEM_NOTRAP_p from rtx_addr_can_trap_
From: tschwinge
gcc/c/
* c-parser.c (c_parser_oacc_all_clauses): Handle
PRAGMA_OMP_CLAUSE_IF.
(OACC_DATA_CLAUSE_MASK, OACC_KERNELS_CLAUSE_MASK)
(OACC_PARALLEL_CLAUSE_MASK, OACC_UPDATE_CLAUSE_MASK): Add it.
gcc/
* omp-low.c (scan_sharing_clau
From: tschwinge
--enable-target-gcc-configure-flags, EXTRA_TARGET_GCC_FLAGS vs.
--enable-accelerator-gcc-configure-flags, EXTRA_ACCELERATOR_GCC_FLAGS.
* configure.ac (--enable-target-gcc-configure-flags)
(--enable-accelerator-gcc-configure-flags): New configure options.
*
It seems the https transition broke refering to permanently moved
URL gcc-3.0/gcc-3.0.html (I get a certificate error or some such),
breaking gennews and thus gcc_release. Fixed like below which
makes gennews succeed.
Committed to the 4.7 branch.
Richard.
2014-06-12 Richard Biener
The GCC 4.7 branch is now closed, please refrain from committing anything
there now.
Richard.
Hi all,
we have merged the gcc-4_9-branch into linaro/gcc-4_9-branch up to
revision 211054 as r211495. We have also backported this set of revisions:
r209419 as r211497 : PR rtl-optimization/60663
r209457 as r211496 : TRY_EMPTY_VM_SPACE Change aarch64 ilp32
r209559 as r211498 : [AArch64] vrnd<*>
Rainer Orth writes:
> Hi Honza,
>
>>> Unfortunately, AIX isn't the only target massively affected by your
>>> recent patches. This all started with r210597
>>>
>>> 2014-05-17 Jan Hubicka
>>>
>>> * tree-pass.h (make_pass_ipa_comdats): New pass.
>>> * timevar.def (TV_IPA_COMDATS):
Adds additional checks and tests for ill-formed programs.
2014-06-12 Andrew Sutton
* gcc/cp/parser.c (cp_check_type_concept): New.
(cp_check_concept_name): Remove redundant condition from check.
Diagnose misuse of non-type concepts in constrained type specifiers.
This patch add predicate for storewb_pair/loadwb_pair, because aarch64
register pair push and pop instructions only accept constant offset
within certain range.
OK for trunk?
Thanks.
gcc/ChangeLog:
2014-06-12 Renlin Li
* config/aarch64/aarch64.c (offset_7bit_signed_scaled_p): Rename to
On 12 June 2014 04:31, Mike Stump wrote:
> On Jun 10, 2014, at 3:03 PM, Ramana Radhakrishnan
> wrote:
>> I am a bit ambivalent between getting folks to add scan-assembler
>> tests here and worrying between this and getting the behaviour
>> correct. Additionally if you add the complexity of scann
On 06/11/2014 02:00 PM, Christian Bruel wrote:
> On 06/11/2014 06:17 AM, Joern Rennecke wrote:
Joern, is this new target macro interface OK with you ?
>> Yes, this interface should allow me to do switches between rounding
>> and truncating
>> floating-point modes with an add/subtract immediate
On 12 June 2014 01:23, Peter Bergner wrote:
> On Wed, 2014-06-11 at 23:07 +, Joseph S. Myers wrote:
>> On Wed, 11 Jun 2014, Peter Bergner wrote:
>>
>> > I'd like to ping the following patch that fixes PR57653. This did
>> > bootstrap and regtest with no regressions on powerpc64-linux.
>> >
>>
On 05/25/2014 11:35 AM, Richard Sandiford wrote:
Bernd Schmidt writes:
On 02/13/2014 10:18 AM, Richard Sandiford wrote:
contrib/
* dg-extract-results.py: New file.
* dg-extract-results.sh: Use it if the environment seems suitable.
I'm now seeing the following:
Traceback (mos
>
> That is, have a tree_may_be_mergeable_p (), call it during the DFS
> walk storing it alongside the visited edges and thus obtain a result
> for each SCC, stream that as a flag (a special hash value is ugly,
> but well ... I guess it works). The important part is to make an SCC
> !tree_may_be_
Hi,
in this old bug Ian complained that the diagnostic we provide for:
typedef void v;
typedef v (*pf)(v);
is rather unfriendly, especially for people coming from C:
33101.C:2:17: error: ‘’ has incomplete type
33101.C:2:18: error: invalid use of ‘v’
thus Gaby (and Ian) suggested something alo
Hello,
I updated i386.md part of the patch. Initial patch included handling
of blockage, which is obviously superflous. Additionally I merged
32-bit and 64-bit peephole2 versions by using mode-specifier W.
ChangeLog
2014-06-12 Kai Tietz
* config/i386/i386.md (peehole2): To combine
i
> On Thu, Jun 12, 2014 at 10:47 AM, Jan Hubicka wrote:
> > Richard,
> > as briefly discussed before, I would like to teach LTO type merging to not
> > merge
> > types that was declared in anonymous namespaces and use C++ ODR type names
> > (stored in DECL_ASSEMBLER_NAME of the TYPE_DECL) to break
On Jun 12, 2014, at 8:53 AM, Bernd Schmidt wrote:
> I've recently been trying to add ada to my set of tested languages, and I now
> encounter the following:
>
> File "../../git/gcc/../contrib/dg-extract-results.py", line 242, in parse_run
>line = file.readline()
> File "/usr/lib64/python3.
On Jun 12, 2014, at 7:26 AM, Christophe Lyon wrote:
> On 12 June 2014 04:31, Mike Stump wrote:
>> On Jun 10, 2014, at 3:03 PM, Ramana Radhakrishnan
>> wrote:
>>> At this point I'm going to wait to see if any of the testsuite
>>> maintainers step in and comment
>>
>> [ ducks ] So, I wasn’t goin
Hi,
The patch fixes a bug in the AArch64 backend in calculating the
beginning address of the unnamed incoming arguments on the stack, i.e.
the initial value of __va_list->__stack. aarch64_layout_arg incorrectly
calculates the size of named arguments on stack using the number of
registers nee
It was reported that mysql was failing its testsuite due to a regex
routine being mis-compiled on the ppc and s390 platforms. Upon
investigation it was found that the fix for PR61009 was incomplete.
The fix for 61009 changed thread_through_normal_block to return a
tri-state with negative va
Hi!
Seems http://gcc.gnu.org/r197156 effectively reverted
the PR45078 fix for arm*-linux* (where unfortunately tm_file
is always overridden).
Was the removal of vxworks-dummy.h from that line intentional
or just some mistake?
Seems one can't build gcc plugins on arm because of this,
because arm.
These two patches fix multiple ICE that occurred due to DFE being
recently enabled after AutoFDO LIPO linking.
Passes regression and internal testing. Ok for Google/4_8?
Teresa
2014-06-12 Teresa Johnson
Dehao Chen
Google ref b/15521327.
* cgraphclones.c (cgraph
A bit belated, I have now committed the patch as Rev. 211587.
Thanks for confirming that it now works!
Tobias
Bernd Schmidt wrote:
On 06/04/2014 10:36 PM, Tobias Burnus wrote:
Bernd Schmidt wrote:
Even with this applied, I'm still seeing similar failures.
I didn't claim that the patch woul
On Thu, Jun 12, 2014 at 06:21:32PM +0200, Kai Tietz wrote:
> with addition of adding a second peephole2 pass after sched2 pass, I
> was able to get some improvement for PR target/39284. I think by this
> addition we can close bug as fixed.
> Additionally additional peephole2 pass shows better resu
On 06/10/14 01:42, Ilya Enkovich wrote:
Hi,
This patch fixes PR61446. The problem appears when we insert value copies
after transformations. We use the widest extension mode met in a chain, but it
may be wider than original destination register size. This patch checks it and
use smaller mod
On 06/09/2014 04:46 PM, Jason Merrill wrote:
I'm updating -Wabi to allow for warnings about changes between a
previous ABI version and the currently selected one, and rather than
adjust all the warnings for -fabi-version=1 I'd like to tear it out.
Here's a revised patch that I'm checking in.
... in terms of code proper, the below is much better, IMHO. Assuming,
as I understand, we have no reason to call the rather heavy same_type_p
when we already know that VOID_TYPE_P (type) is true...
Thanks,
Paolo.
//
Index: cp/decl.c
I talked about doing this in 4.9
(https://gcc.gnu.org/ml/gcc/2013-03/msg8.html), but decided to put
it off along with the libstdc++ ABI transition. I think it's time now.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit a2aa0efcd1f27e85a4c652f5177c66686f530a96
Author: Jason Merrill
D
On Thu, Jun 05, 2014 at 10:43:25PM +0100, Richard Sandiford wrote:
> This final patch uses a common .md file to define all standard
> constraints except 'g'.
I had a look at what targets still use "g". Note: there can be
errors in this, it's all based on \
On Jun 12, 2014, at 12:17 PM, Jason Merrill wrote:
> I talked about doing this in 4.9
> (https://gcc.gnu.org/ml/gcc/2013-03/msg8.html), but decided to put it off
> along with the libstdc++ ABI transition. I think it's time now.
Is a doc change needed?
> @opindex fabi-version
> Use version
On Jun 12, 2014, at 3:24 PM, Segher Boessenkool
wrote:
> On Thu, Jun 05, 2014 at 10:43:25PM +0100, Richard Sandiford wrote:
>> This final patch uses a common .md file to define all standard
>> constraints except 'g'.
>
> I had a look at what targets still use "g". Note: there can be
> errors
How does this affect pr60732?
Dominique
On 06/10/14 01:42, Ilya Enkovich wrote:
Hi,
This patch fixes PR61446. The problem appears when we insert value copies
after transformations. We use the widest extension mode met in a chain, but it
may be wider than original destination register size. This patch checks it and
use smaller mod
On 06/12/14 04:12, Richard Biener wrote:
This implements the requested inlining of memmove for possibly
overlapping arguments by doing first all loads and then all stores.
The easiest place is to do this in memory op folding where we already
perform inlining of some memcpy cases (but fail to do
2014-06-12 20:52 GMT+02:00 Segher Boessenkool :
> On Thu, Jun 12, 2014 at 06:21:32PM +0200, Kai Tietz wrote:
>> with addition of adding a second peephole2 pass after sched2 pass, I
>> was able to get some improvement for PR target/39284. I think by this
>> addition we can close bug as fixed.
>> Ad
Add support for partial concept ids. Mostly this just refactors the
basic support for concept names to also allow a template and extra
arguments.
Also added the missing .exp file for the test suite.
2014-06-12 Andrew Sutton
* gcc/cp/constraint.cc (deduce_constrained_parameter): Refacto
Now that -fabi-version defaults to 0, -Wabi isn't very useful. But for
people interested in compatibility with earlier versions, this patch
allows you to say -Wabi=2 to get any relevant warnings. This patch also
adjusts the compatibility aliases to default to backward compatibility
with -fabi
On 06/12/2014 03:36 PM, Mike Stump wrote:
On Jun 12, 2014, at 12:17 PM, Jason Merrill wrote:
I talked about doing this in 4.9
(https://gcc.gnu.org/ml/gcc/2013-03/msg8.html), but decided to put it off
along with the libstdc++ ABI transition. I think it's time now.
Is a doc change needed
On 06/12/2014 03:44 PM, Dominique Dhumieres wrote:
How does this affect pr60732?
It should fix that failure.
Jason
> > Will that work on other targets?
> Well, this is the only point I am a bit concerned too. In general I
> wouldn't expect here any issues to run peephole after scheduling, as
> peephole doesn't do anything a new run of ira/lra would require.
My concern is that peepholes are rather fragile, so
Hi!
This patch fixes 3 issues:
1) distribute doesn't support lastprivate clause, so gimplification
shouldn't add it, it causes ICEs
2) for shared clauses on teams construct we need to at least
record something in decl_map, otherwise lookup_decl ICEs
3) c_omp_split_clauses ICEd on a couple of
> > * cris, m68k, pdp11, and vax actually use "g".
> >
> > So it won't be all that much work to completely get rid of "g".
> > Do we want that?
>
> Is it simply a matter of replacing “g” by “mri”? That’s what the doc
> suggests. Or is there more to the story than that?
As far as I know "g" an
On 06/12/2014 03:14 PM, Paolo Carlini wrote:
... in terms of code proper, the below is much better, IMHO. Assuming,
as I understand, we have no reason to call the rather heavy same_type_p
when we already know that VOID_TYPE_P (type) is true...
same_type_p is not so heavy since it just compares
On 6/12/2014 9:21 AM, Kai Tietz wrote:
with addition of adding a second peephole2 pass after sched2 pass, I
was able to get some improvement for PR target/39284. I think by this
addition we can close bug as fixed.
Additionally additional peephole2 pass shows better results for PR
target/51840 t
I think the patch looks good. David and Rong, any comments?
Dehao
On Thu, Jun 12, 2014 at 11:23 AM, Teresa Johnson wrote:
> These two patches fix multiple ICE that occurred due to DFE being
> recently enabled after AutoFDO LIPO linking.
>
> Passes regression and internal testing. Ok for Google/4
Hi,
while updating vect_can_force_dr_alignment_p for section API I noticed the
predicate is bit confused about when it can update the alignment.
We need to check that decl_binds_to_current_def_p and in case we compile
a partition also that the symbol is not homed in other partition.
Previous code
DR1206 allows explicit specializations of member enumerations
of class templates, so just remove the pedwarn about it.
Tested on Linux-x64. Not bootstrapped.
diff --git a/gcc/cp/pt.c b/gcc/cp/pt.c
index d267a5c..97eadeb 100644
--- a/gcc/cp/pt.c
+++ b/gcc/cp/pt.c
@@ -967,11 +967,8 @@ maybe_process_
On 13 June 2014 01:37, Ville Voutilainen wrote:
> DR1206 allows explicit specializations of member enumerations
> of class templates, so just remove the pedwarn about it.
>
> Tested on Linux-x64. Not bootstrapped.
Argh, also remove the old comment, new patch attached.
diff --git a/gcc/cp/pt.c b/g
This looks fine to me.
-Rong
On Thu, Jun 12, 2014 at 11:23 AM, Teresa Johnson wrote:
> These two patches fix multiple ICE that occurred due to DFE being
> recently enabled after AutoFDO LIPO linking.
>
> Passes regression and internal testing. Ok for Google/4_8?
>
> Teresa
>
> 2014-06-12 Teresa
2014-06-12 11:55 GMT+04:00 Richard Biener :
> On Wed, Jun 11, 2014 at 6:08 PM, Ilya Enkovich wrote:
>> Hi,
>>
>> This patch fixes problem with size emitted for static structures with
>> flexible array. I found a couple of trackers in guzilla for this problem
>> but all of them are marked as fix
> >
> > Comdat locals are now used by ipa-comdats, for thunks and for decloned
> > ctors.
> > We probably need to figure out bit more precise limitation of Solaris and
> > either
> > fix or add way for target to say what kind of comdat locals are not
> > supported.
>
> Right. I'll start reghun
>
> When you extract the address and use it. For example when you
> do auto-parallelization and outline a part of your function it
> passes arrays as addresses.
>
> Or if you start to introduce address induction variables like
> the vectorizer or IVOPTs does.
I see, nothing really done by curre
On Mon, 12 May 2014, Tobias Burnus wrote:
The patch changes the URL shown in the release message to HTTPS. (Cf.
https://gcc.gnu.org/viewcvs/gcc/hooks/svnmailer.conf and gcc-cvs mailing
list.)
Yes, please. Thanks!
Gerald
78 matches
Mail list logo
