On Mon, Jun 04, 2012 at 11:19:46PM -0700, Ian Lance Taylor wrote:
> This patch to libgo includes the TLS size in the requested stack size of
> a new thread, if possible. This relies on the glibc-specific (and
> undocumented) _dl_get_tls_static_info call. This is particularly
> necessary when usin
Dimitrios Apostolou writes:
> Hi Dodji,
>
> On Mon, 4 Jun 2012, Dodji Seketeli wrote:
>
>> Hello Dimitrios,
>>
>> I cannot approve or deny your patch, but I have one question.
>>
>
> Who should I CC then?
Do not worry, I am CC-ing the maintainers. I just forgot to CC them
when I replied. Sorr
On Mon, Jun 4, 2012 at 8:23 PM, Steven Bosscher wrote:
> Hello,
>
> The attached patch removes one more #include output.h, this time from
> c-family/c-pch.c.
>
> Anything written out to asm_out_file between pch_init and
> c_common_write_pch is read back in by c_common_write_pch and dumped to
> the
On Mon, Jun 4, 2012 at 9:07 PM, Marc Glisse wrote:
> On Mon, 4 Jun 2012, Florian Weimer wrote:
>
>> On 06/01/2012 01:34 PM, Jakub Jelinek wrote:
>>>
>>> Have you looked at the assembly differences with this in?
>>
>>
>> It's not great.
>>
>> Here's an example:
>>
>> void
>> write(std::vector& blob
Hi
This is the fourth part of the patches that deals with 64bit ior. It directly
extends the patterns iordi3, iordi3_insn and iordi3_neon to handle 64bit
constant operands.
Tested on arm qemu without regression.
OK for trunk?
thanks
Carrot
2012-06-05 Wei Guozhi
PR target/53447
On Tue, Jun 5, 2012 at 9:20 AM, Jakub Jelinek wrote:
> On Mon, Jun 04, 2012 at 11:19:46PM -0700, Ian Lance Taylor wrote:
>> This patch to libgo includes the TLS size in the requested stack size of
>> a new thread, if possible. This relies on the glibc-specific (and
>> undocumented) _dl_get_tls_st
On Thu, 31 May 2012, Richard Guenther wrote:
> On Wed, 30 May 2012, Richard Guenther wrote:
>
> >
> > The patch below extents memset recognition to cover a few more
> > non-byte-size store loops and all byte-size store loops. This exposes
> > issues with our builtins.exp testsuite which has cus
Hi Alessandro,
I am glad to see that Janus is giving you a helping hand, in addition
to Tobias. I am so tied up with every aspect of life that gfortran is
not figuring much at all.
When you clean up the patch, you might consider making this into a
separate function:
+ if (free_proc)
+
The GCC 4.7 branch is now frozen for creating a first release candidate
of the GCC 4.7.1 release.
All changes need explicit release manager approval until the final
release of GCC 4.7.1 which should happen roughly one week after
the release candidate if no issues show up with it.
Previous Repo
Well - this replaces passing down a flag pointer with a new member
in the partition struct.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2012-06-05 Richard Guenther
* tree-loop-distribution.c (struct partition_s): Add has_writes
member.
(par
On Mon, Jun 4, 2012 at 8:09 PM, H.J. Lu wrote:
> We should enable x32 run-time library if --with-abi={x32|mx32} is used
> to configure GCC i[34567]86-*-* and x86_64-*-*. Tested on Linux/x86-64.
> OK for trunk?
>
> 2012-06-04 H.J. Lu
>
> PR target/53575
> * config.gcc: Enable x32
I've merged gcc-4_7-branch revision 188231 to the gccgo branch.
Ian
PR30442 shows that we do not vectorize basic-blocks if the to-be
vectorized data-references are followed by something that
find_data_references_in_stmt does not know how to analyze
(any call or asm for example). The following re-organizes how
we create data-references in vect_analyze_data_refs fo
On Tue, Jun 5, 2012 at 5:24 AM, Uros Bizjak wrote:
> On Mon, Jun 4, 2012 at 8:09 PM, H.J. Lu wrote:
>
>> We should enable x32 run-time library if --with-abi={x32|mx32} is used
>> to configure GCC i[34567]86-*-* and x86_64-*-*. Tested on Linux/x86-64.
>> OK for trunk?
>>
>> 2012-06-04 H.J. Lu
On Tue, Jun 5, 2012 at 2:47 PM, H.J. Lu wrote:
>>> We should enable x32 run-time library if --with-abi={x32|mx32} is used
>>> to configure GCC i[34567]86-*-* and x86_64-*-*. Tested on Linux/x86-64.
>> Why all three ABIs here? Didn't user specify -with-abi=mx32 only, so
>> x86_multilibs="mx32" o
Jakub Jelinek writes:
> On Mon, Jun 04, 2012 at 11:19:46PM -0700, Ian Lance Taylor wrote:
>> This patch to libgo includes the TLS size in the requested stack size of
>> a new thread, if possible. This relies on the glibc-specific (and
>> undocumented) _dl_get_tls_static_info call. This is parti
On Thu, May 31, 2012 at 8:10 PM, wrote:
> Reviewers: xur, davidxl, iant2, Diego Novillo,
>
> Message:
> The relevant bug is this:
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53546.
This bug does not affect trunk. Only the google branches where LIPO
is implemented.
Diego.
On Tue, Jun 05, 2012 at 02:35:30PM +0200, Richard Guenther wrote:
> Index: gcc/tree-vect-data-refs.c
> ! gimple stmt = gsi_stmt (gsi);
> ! if (!find_data_references_in_stmt (NULL, stmt,
> ! &BB_VINFO_DATAREFS (bb_vinfo)))
> ! {
> !
On 12-06-04 23:08 , Sandeep Soni wrote:
Hi,
I have added some more preliminary tests for the gimple front end..
These are really simple individual statement tests which have helped
me get a hang of the testing mechanism in gcc. I will be adding a few
more tests tonight.
Sounds good.
At this
Il 04/06/2012 11:31, Richard Guenther ha scritto:
> +val = compare_range_with_value (NE_EXPR, vr, integer_zero_node,
> &sop);
> +if (!val || !integer_onep (val))
> + return false;
>
> please add a value_range_nonzero_p helper alongside value_range_nonnegative_p.
>
> +
On Tue, 5 Jun 2012, Nathan Froyd wrote:
> On Tue, Jun 05, 2012 at 02:35:30PM +0200, Richard Guenther wrote:
> > Index: gcc/tree-vect-data-refs.c
> > ! gimple stmt = gsi_stmt (gsi);
> > ! if (!find_data_references_in_stmt (NULL, stmt,
> > !&BB_VINFO_D
On Mon, Jun 4, 2012 at 3:29 PM, Sriraman Tallam wrote:
> Bug fixed and new patch attached.
>
> Patch also available for review at http://codereview.appspot.com/5752064
>
I think you should also export __cpu_indicator_init in libgcc_s.so.
Also, is this feature C++ only? Can you make it to work fo
On 06/05/2012 12:14 AM, Mike Stump wrote:
On Jun 4, 2012, at 7:46 PM, Mike Stump wrote:
g++.dg/other/warning1.C -std=c++11 (test for warnings, line 10)
g++.dg/other/warning1.C -std=c++11 (test for warnings, line 11)
g++.dg/other/warning1.C -std=c++11 (test for excess errors)
g++.dg/other/warni
On 06/05/2012 07:00 AM, Paolo Carlini wrote:
(construct_virtual_base): Adjust LOOKUP_COMPLAIN -> LOOKUP_NORMAL.
This and the similar changes elsewhere seem dangerous; they're adding
adding LOOKUP_PROTECT that wasn't there before. Instead, let's replace
LOOKUP_COMPLAIN with 0 or some
On Tue, 5 Jun 2012, Richard Guenther wrote:
> On Tue, 5 Jun 2012, Nathan Froyd wrote:
>
> > On Tue, Jun 05, 2012 at 02:35:30PM +0200, Richard Guenther wrote:
> > > Index: gcc/tree-vect-data-refs.c
> > > ! gimple stmt = gsi_stmt (gsi);
> > > ! if (!find_data_references_in_stmt
Applied, thanks.
Jason
This adds memcpy/memmove recognition to loop distribution (and
cleans it up some more). Issues are similar to memset and
not handled (and I just noticed we generate memset/memcpy even
with -fno-builtin ...).
Bootstrapped on x86_64-unknown-linux-gnu, testing in progress.
Richard.
2012-06-05 Ri
Richard Sandiford wrote:
> Ian Lance Taylor writes:
> > Richard Sandiford writes:
> >> gcc/
> >>* doc/invoke.texi (sched-pressure-algorithm): Document new --param.
> >>* common.opt (fsched-pressure-algorithm=): Remove.
> >>* flag-types.h (sched_pressure_algorithm): Move to...
> >>
Hi,
what is the blocking point for the integration of these patches?
--
Sebastian Huber, embedded brains GmbH
Address : Obere Lagerstr. 30, D-82178 Puchheim, Germany
Phone : +49 89 18 90 80 79-6
Fax : +49 89 18 90 80 79-9
E-Mail : sebastian.hu...@embedded-brains.de
PGP : Public key a
On 06/04/2012 10:46 PM, Jason Merrill wrote:
On 06/04/2012 04:12 PM, Florian Weimer wrote:
This doesn't make sense to me. parser->integral_constant_expression_p
should always be true at this point if you're moving the restore later
(which also seems unnecessary).
I think parser->integral_const
On 05/14/2012 08:51 PM, Joseph S. Myers wrote:
On Mon, 14 May 2012, Joel Sherrill wrote:
There is a long explanation in the PR but the short
version is that although we fully intended to switch
the arm-rtems target from ELF to EABI we never
intended the target name "arm-*-rtemseabi*" to
become
Hi,
Il giorno 05/giu/2012, alle ore 16:16, Jason Merrill ha
scritto:
> On 06/05/2012 07:00 AM, Paolo Carlini wrote:
>>(construct_virtual_base): Adjust LOOKUP_COMPLAIN -> LOOKUP_NORMAL.
>
> This and the similar changes elsewhere seem dangerous; they're adding adding
> LOOKUP_PROTECT that
I committed the following patch to fix the dates
on my own ChangeLog entries
Thanks
Edmar
Index: gcc/ChangeLog
===
--- gcc/ChangeLog (revision 188244)
+++ gcc/ChangeLog (working copy)
@@ -1,4 +1,4 @@
-2012-06-01 Edmar
Hello Everyone,
This patch is for the Cilkplus branch affecting the C++ compiler. This
patch will fix some cases were array notation is not decomposed correctly in
parser.
Thanking You,
Yours Sincerely,
Balaji V. Iyer.
Index: gcc/cp/parser.c
On 06/05/2012 11:29 AM, Paolo Carlini wrote:
To be sure: NORMAL used to be just PROTECT | COMPLAIN, thus it's just about
names, right?
Yes.
You mean, we do away with the NORMAL name, you mean?
We could, but I think it's fine to have it as an alias for
LOOKUP_PROTECT; the LOOKUP_NORMAL nam
My colleagues and I have been working on the GCC port for the Qualcomm
Hexagon. Along the way I noticed that we were getting poor results
from the ivopts pass no matter how we adjusted the target-specific RTX
costs. In many cases ivopts was coming up with candidate use costs
that seemed completel
Hi,
> We could, but I think it's fine to have it as an alias for LOOKUP_PROTECT;
> the LOOKUP_NORMAL name implies that we're doing a normal name lookup, whereas
> LOOKUP_PROTECT is what that implies.
Believe it or not, yesterday for a few minutes I had it exactly as an alias.
Ok, I'll do that.
On Fri, May 18, 2012 at 5:05 PM, Eric Botcazou wrote:
> * configure.ac (HAVE_GNU_LD): Move to after config.gcc inclusion.
> (HAVE_GNU_AS): Likewise.
> * config.in: Regenerate.
> * configure: Likewise.
Eric,
This change is breaking bootstraps in one of the google bran
On 06/05/2012 11:19 AM, Florian Weimer wrote:
True. But if we want cascading errors, cp_parser_constant_expression
really cannot return error_mark_node, so this approach is a dead end.
(For example, build_enumerator replaces error_mark_node in the
enumeration value with nothing, i.e., the next po
On Tue, Jun 5, 2012 at 12:05 PM, Jason Merrill wrote:
> In the operator new case, the non-constant expression error followed by the
> VLA warning is not as helpful, as the latter ignores the former. Perhaps the
> right way to deal with this is to allow non-constant expressions in the
> new-type-i
OK.
Jason
http://codereview.appspot.com/6282045/diff/1/gcc/gcov-io.h
File gcc/gcov-io.h (right):
http://codereview.appspot.com/6282045/diff/1/gcc/gcov-io.h#newcode544
gcc/gcov-io.h:544: gcov_unsigned_t sum_cutoff_percent;/* sum_all cutoff
percentage computed
Is there a need to record this?
http://coderev
The following fixes a problem with my recently added testcase that resulted in
failure for 32-bit since instructions to stack a frame reduced the number of
nop's that were needed to force the load into a separate dispatch group.
Tested on powerpc64-linux, committed as obvious.
-Pat
testsuite
Hi Lawrence,
On Mon, 4 Jun 2012, Lawrence Crowl wrote:
> The following source change enables coddingconventions.html to
> pass the HTML validator at validator.w3.org.
the web pages will be preprocessed before the are put on the server
(this transparently happens upon checkin) and as part of that
Hello,
This patch just cleans up some remaining code for removed cris-aout
subtarget by folding away code that was conditional on TARGET_ELF.
Tested with a x86_64-linux X cris-elf cross-compiler. OK for trunk?
Ciao!
Steven
cris_small_cleanups.diff
Description: Binary data
On May 23, 2012, Jakub Jelinek wrote:
> On Wed, May 23, 2012 at 06:27:21AM -0300, Alexandre Oliva wrote:
>> + for (loc = var->var_part[0].loc_chain; loc; loc = loc->next)
>> +if (GET_CODE (loc->loc) == MEM
>> +&& !nonoverlapping_memrefs_p (loc->loc, mloc, false))
> Isn't
The patch I submitted had an omission. I failed to regenerate
rs6000-tables.opt
(Sorry, I misunderstood gcc_update --touch instructions)
OK to commit the update ?
2012-06-05 Edmar Wienskoski
* config/rs6000/rs6000-tables.opt: Regenerated.
On 06/04/2012 08:45 PM, David Edelsohn wrote:
T
On 6/5/12, Gerald Pfeifer wrote:
> On Mon, 4 Jun 2012, Lawrence Crowl wrote:
> > The following source change enables coddingconventions.html to
> > pass the HTML validator at validator.w3.org.
>
> the web pages will be preprocessed before the are put on the server
> (this transparently happens upo
David, Michael,
Here is the new type "popcnt" patch that I had separated from previous
E5500/E6500 submission, also added the changes suggested by Michael
Meissner (detailed bellow).
I am missing some details for power6. (Could not find any documentation)
Bootstrapped with no regressions, all la
I committed the following patch to the FSF 4.6 branch which Jakub
approved on IRC.
Peter
Backport from mainline
2011-08-29 Jakub Jelinek
* gthr-posix.h (__gthread_active_p): Do not use preprocessor
conditionals and comments inside macro arguments.
Index: gcc/
Hi H.J.,
I am attaching a patch to add __cpu_indicator_init to the list of
symbols to be versioned and exported in libgcc_s.so. Also, updating
builtin_target.c test to explicitly do a CPUID and check if the
features are identified correctly like you had suggested earlier.
Patch ok?
* co
ping ? is there any problem with that patch ?
Hello Everyone,
This patch is for the Cilkplus branch affecting the C compiler. This patch
will remove an unwanted conversion of the induction variable in the cilk_for
function.
Thanking You,
Yours Sincerely,
Balaji V. Iyer.Index: gcc/cilk-spawn.c
=
On Jun 5, 2012, at 6:46 AM, Paolo Bonzini wrote:
>> Do we always have CTZ if we have FFS? Can't there be a target that
>> implements FFS as opcode but not CTZ, so you'd slow down things?
>> Thus, should the transform be conditonal on target support for CTZ
>> or no target support for FFS?
>
> H
On 06/05/2012 08:23 PM, Paolo Carlini wrote:
@@ -1695,6 +1695,8 @@ implicit_conversion (tree to, tree from, tree expr
|LOOKUP_NO_TEMP_BIND|LOOKUP_NO_RVAL_BIND|LOOKUP_PREFER_RVALUE
|LOOKUP_NO_NARROWING|LOOKUP_PROTECT);
+ complain&= ~tf_error;
I don't think we want warni
Patch updated: using regex to match the function name:
http://codereview.appspot.com/6281047
Thanks,
Dehao
2012-06-01 Dehao Chen
* gcc/cgraph.c (cgraph_node): Add attribute to function decl.
* gcc/opts-global.c (add_attribute_pattern): New function.
(pattern_match_fun
Please document it in doc/invoke.texi with examples.
thanks,
David
On Tue, Jun 5, 2012 at 7:28 PM, Dehao Chen wrote:
> Patch updated: using regex to match the function name:
>
> http://codereview.appspot.com/6281047
>
> Thanks,
> Dehao
>
> 2012-06-01 Dehao Chen
>
> * gcc/cgraph.c (cgr
Also needs to get the attribute spec and call the attribute handler ..
David
On Tue, Jun 5, 2012 at 9:28 PM, Xinliang David Li wrote:
> Please document it in doc/invoke.texi with examples.
>
> thanks,
>
> David
>
> On Tue, Jun 5, 2012 at 7:28 PM, Dehao Chen wrote:
>> Patch updated: using regex
58 matches
Mail list logo
