On Thu, 29 Mar 2012, Jan Hubicka wrote:
> >
> > I am playing with doing some internal interface static analysis
> > using the first patch below (and looking at LTO bootstrap results).
> >
> > An example, obvious patch resulting from that is the 2nd patch,
> > resuling from the static analysis ou
On Fri, 30 Mar 2012, Martin Jambor wrote:
> Hi,
>
> when testing a patch of mine on sparc64-linux, I came across an Ada
> bootstrap failure due to a structure DECL which was marked addressable
> but had a register DECL_RTL (and therefore mem_ref_refers_to_non_mem_p
> failed to trigger on it).
>
On 29 March 2012 22:10, Kenneth Zadeck wrote:
> This patch takes a different approach to fixing PR52543 than does the patch
> in
>
> http://gcc.gnu.org/ml/gcc-patches/2012-03/msg00641.html
>
> This patch transforms the lower-subreg pass(es) from unconditionally
> splitting wide moves, zero extensi
Hi,
up to now, the CRTL map (map between C standard function name and their
corresponding name in the DEC-C shared library) was manually written: entries
were added when needed.
I take a radical approach, using nm on this library (and the math one, DPML) to
extract all symbols, wrote a script
Dodji Seketeli a écrit:
> Hello,
>
> I forgot to update changes.html when the -ftrack-macro-expansion and
> Wunused-local-typedefs changes went in.
>
> Fixed thus. I hope it is not too late, now that 4.7 is out.
>
> OK for CVS head?
>
> * htdocs/gcc-4.7/changes.html: Update for
>
Last one (for now).
Bootstrapped on x86_64-unknown-linux-gnu, applied.
Richard.
2012-03-30 Richard Guenther
* tree-affine.h (print_aff): Remove.
* tree-affine.c (print_aff): Make static.
* tree-data-ref.h (access_matrix_get_index_for_parameter): Remove.
(get_
In some cases, a node designating a compilation unit may be empty, which was
not considered in the code generating Alfa sections in ALI files. Now corrected.
Tested on x86_64-pc-linux-gnu, committed on trunk
2012-03-30 Yannick Moy
* lib-xref-alfa.adb (Add_Alfa_File): Take into account
When the declaration of the loop entity of an Ada-2012-style array iterator
is rewritten as a renaming of the indexed array, debug info was not being
generated for the renaming, preventing display of the entity (gdb generates
a "no definition in current context" message). The loop entity of such a
This fixes PR52786 which I did not see in my testing (huh). I suppose
hppa*-*-* has unsigned HOST_WIDE_INT == unsigned int and we suppress
the sign-compare warning for unsigned long >= (long) unsigned int
Committed as obvious.
Richard.
2012-03-30 Richard Guenther
PR middle-end/5278
This patch updates the mechanism which detects build-in-place function calls
returning controlled results on the secondary stack.
-- Source --
-- types.ads
with Ada.Finalization; use Ada.Finalization;
package Types is
type Ctrl_Comp is new Limited_Controlled with nu
This reformatting is meant to clarify the code generating Alfa cross-references
so that it can be updated to take into account instantiations.
Tested on x86_64-pc-linux-gnu, committed on trunk
2012-03-30 Yannick Moy
* lib-xref-alfa.adb, lib-xref.adb: Code clean ups.
Index: lib-xref-a
In order to apply formal verification to code with instantiations of generics,
we must get Alfa cross-references for this code. This consists mostly in
enabling the generation of instances in Alfa mode, and adapting the Alfa xref
code.
Tested on x86_64-pc-linux-gnu, committed on trunk
2012-03-30
This patch corrects the preanalysis of quantified expressions to avoid the
generation of temporaries to capture the bounds of iteration.
-- Source --
-- main.adb
with Ada.Text_IO; use Ada.Text_IO;
procedure Main is
Bound : Integer;
function F return Integer is
This patch updates the mechanism which detects build-in-place function calls
returning controlled results on the secondary stack.
-- Source --
-- types.ads
with Ada.Finalization; use Ada.Finalization;
package Types is
type Ctrl_Comp is new Limited_Controlled with nu
Kenneth Zadeck writes:
> This patch takes a different approach to fixing PR52543 than does the
> patch in
>
> http://gcc.gnu.org/ml/gcc-patches/2012-03/msg00641.html
>
> This patch transforms the lower-subreg pass(es) from unconditionally
> splitting wide moves, zero extensions, and shifts, so t
On Mar 27, 2012, at 10:38 AM, Janne Blomqvist wrote:
> On Tue, Mar 27, 2012 at 11:01, Tristan Gingold wrote:
>> Hi,
>>
>> this patch fixes this issue. Is it OK ?
>
> Ok.
>
>> Maybe we should include the AC_DEFINE action within GCC_CHECK_MATH_FUNC.
>> Will try to do that.
>
> That looks li
http://gcc.gnu.org/ml/gcc-patches/2012-03/msg01269.html
Ping.
--
Dr. Ulrich Weigand
GNU Toolchain for Linux on System z and Cell BE
ulrich.weig...@de.ibm.com
Hello!
2012-03-30 Uros Bizjak
PR libgfortran/52758
* intrinsics/chmod.c: Remove out-of-bounds initialization of rwxXstugo.
Bootstrapped and regression tested on x86_64-pc-linux-gnu {,-m32},
committed to mainline SVN as obvious.
Uros.
Index: intrinsics/chmod.c
On 28/02/12 16:20, Andrew Stubbs wrote:
Hi all,
This patch implements 64-bit immediate constant loads in NEON.
The current state is that you can load const_vector, but not const_int.
This is clearly not ideal. The result is a constant pool entry when it's
not necessary.
The patch disables the
> On IRC I've been told that is OK and the that CCP cannot make such
> assumtions. Since it is only a missed-optimization if the call to the
> builtin is not found and processed (basically PR 51491 again but only
> in cases like these), I thought it best to just remove the assert by
> the followin
Hi,
>> Looks that the patch ignores the case using movml.
>> It could be something like the attached patch
Sorry for ignoring the case using movml.
Thanks for the patch which takes care of movml case.
>> though I don't do any tests.
The patch was tested with movml testcase and works as expec
On Fri, 30 Mar 2012, Richard Guenther wrote:
> On Thu, 29 Mar 2012, Jan Hubicka wrote:
>
> > With Mozilla folks I used the dumps from first WPA unreachable function
> > removal pass
> > with some degree of success. This gets a lot of non-trivial cases of dead
> > code,
> > but also there are a
Il 30/03/2012 12:22, Tristan Gingold ha scritto:
>
> On Mar 27, 2012, at 10:38 AM, Janne Blomqvist wrote:
>
>> On Tue, Mar 27, 2012 at 11:01, Tristan Gingold wrote:
>>> Hi,
>>>
>>> this patch fixes this issue. Is it OK ?
>>
>> Ok.
>>
>>> Maybe we should include the AC_DEFINE action within GCC_C
Hi,
On Thu, 29 Mar 2012, Sriraman Tallam wrote:
> +struct __processor_model
> +{
> + /* Vendor. */
> + unsigned int __cpu_is_amd : 1;
> + unsigned int __cpu_is_intel : 1;
> + /* CPU type. */
> + unsigned int __cpu_is_intel_atom : 1;
> + unsigned int __cpu_is_intel_core2 : 1;
> + unsigned i
Hi,
this attribute is used for some specialized Ada constructs available only on
VMS.
The current implementation is flawed when there are two (or more) variables
with the same common_object.
This patch uses the same mechanism as the one used by Alpha VMS.
Manually tested on ia64-hp-openvms.
Co
Hi,
I've re-tested the patch from:
http://gcc.gnu.org/ml/gcc-patches/2011-11/msg01819.html
on s390x and x86_64.
Ok for mainline?
Bye,
-Andreas-
This fixes PR52772 - prev_bb does not have any relation to the new
pre-landing-pad block (not sure what I was thinking here), so this
moves the loop updating code to the place where we connect the
new block into the CFG.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Rich
This fixes another case of bogusly reconstructed array references.
The symptom is that predictive commoning creates negative array
indices (and in a wrong way, too).
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Richard.
2012-03-30 Richard Guenther
PR tree-o
Hi,
this is the minimum required to build gfortran on VMS.
As vms.h defines TARGET_EXTRA_INCLUDES, we need to add an empty replacement for
gfortran not unlike darwin.
Tested by building gfortran.
Committed on trunk.
Tristan.
2012-03-30 Tristan Gingold
* config/vms/vms-f.c: New file
Hi,
Please find the attached patch to avoid saving of EXR register for
monitor functions.
By default, in prologue code of a monitor function, EXR register is
pushed onto the stack. This implementation is not required for H8S/224x
and 21xx variants of H8S controllers. The behavior can be controlle
Pulling this one back as I have a better solution, patch coming shortly.
Thanks,
Teresa
On Fri, Mar 16, 2012 at 3:33 PM, Teresa Johnson wrote:
>
> Ping - now that stage 1 is open, could someone review?
>
> Thanks,
> Teresa
>
> On Sun, Dec 4, 2011 at 10:26 PM, Teresa Johnson wrote:
> > Latest pa
This patch addresses instructions that incur expensive length-changing prefix
(LCP) stalls
on some x86-64 implementations, notably Core2 and Corei7. Specifically, a move
of
a 16-bit constant into memory requires a length-changing prefix and can incur
significant
penalties. The attached patch avo
Hi,
currently all VMS compilers are built on Unix. So, to build the libstdc++
library, this looks like the minimum required.
Tested by build libstdc++ for ia64-hp-openvms.
Ok for trunk ?
Tristan.
libstdc++/
2012-03-30 Tristan Gingold
* crossconfig.m4 (*-*-*vms*): Add.
* c
I should add that I have tested performance of this on Core2, Corei7
(Nehalem) and AMD Opteron-based systems. It appears to be
performance-neutral on AMD (only minor perturbations, overall a wash).
For the test case that provoked the optimization, there were nice
improvements on Core2 and Corei7.
Hi,
there are some systems on which getrlimit/setrlimit are not available, and
compiling stack-limit.c on these systems generates a warning.
Cleaned up with this patch.
Tested while building gcc for ia64-hp-openvms on x86_64-darwin.
Ok for trunk ?
Tristan.
libiberty/
2012-03-30 Tristan Gingo
> This patch takes a different approach to fixing PR52543 than does the
> patch in
>
> http://gcc.gnu.org/ml/gcc-patches/2012-03/msg00641.html
>
> This patch transforms the lower-subreg pass(es) from unconditionally
> splitting wide moves, zero extensions, and shifts, so that it now takes
> in
Minor update to patch to remove unnecessary check in new movhi_imm_internal
define_insn.
Retested successfully.
Teresa
2012-03-29 Teresa Johnson
* config/i386/i386.h (ix86_tune_indices): Add
X86_TUNE_LCP_STALL.
* config/i386/i386.md (movhi_internal): Split to
Mike Stump writes:
>> Here is the new patch. OK for trunk if there are no regressions on
>> Linux/ia32 and Linux/x86-64?
>
> Too bad you didn't test 32-bit darwin, causes:
>
> http://gcc.gnu.org/PR52784
>
> Could you please revert or fix, thanks.
Same problem on Solaris 10 and 11/x86.
On 03/30/2012 11:03 AM, Teresa Johnson wrote:
> +(define_insn "*movhi_imm_internal"
> + [(set (match_operand:HI 0 "memory_operand" "=m")
> +(match_operand:HI 1 "immediate_operand" "n"))]
> + "!TARGET_LCP_STALL"
> +{
> + return "mov{w}\t{%1, %0|%0, %1}";
> +}
> + [(set (attr "type") (con
> Index: config/i386/i386.md
> ===
> --- config/i386/i386.md (revision 185920)
> +++ config/i386/i386.md (working copy)
> @@ -2262,9 +2262,19 @@
> ]
> (const_string "SI")))])
>
> +(define_insn "*movhi_im
On 03/30/2012 11:11 AM, Richard Henderson wrote:
> On 03/30/2012 11:03 AM, Teresa Johnson wrote:
>> +(define_insn "*movhi_imm_internal"
>> + [(set (match_operand:HI 0 "memory_operand" "=m")
>> +(match_operand:HI 1 "immediate_operand" "n"))]
>> + "!TARGET_LCP_STALL"
>> +{
>> + return "mov
+ There are two useful preprocessor defines for use by maintainers:
+
+ #define LOG_COSTS
+
+ if you wish to see the actual cost estimates that are being used
+ for each mode wider than word mode and the cost estimates for zero
+ extension and the shifts. This can be useful when po
On 03/30/2012 10:39 AM, Georg-Johann Lay wrote:
This patch takes a different approach to fixing PR52543 than does the
patch in
http://gcc.gnu.org/ml/gcc-patches/2012-03/msg00641.html
This patch transforms the lower-subreg pass(es) from unconditionally
splitting wide moves, zero extensions, an
Kenneth Zadeck writes:
>>> + There are two useful preprocessor defines for use by maintainers:
>>> +
>>> + #define LOG_COSTS
>>> +
>>> + if you wish to see the actual cost estimates that are being used
>>> + for each mode wider than word mode and the cost estimates for zero
>>> + extensi
"Georg-Johann Lay" writes:
>> This patch takes a different approach to fixing PR52543 than does the
>> patch in
>>
>> http://gcc.gnu.org/ml/gcc-patches/2012-03/msg00641.html
>>
>> This patch transforms the lower-subreg pass(es) from unconditionally
>> splitting wide moves, zero extensions, and
Tristan Gingold writes:
> 2012-03-30 Tristan Gingold
>
> * stack-limit.c: Includes ansidecl.h.
> (stack_limit_increase): Add ATTRIBUTE_UNUSED
This is OK.
Thanks.
Ian
On 03/21/2012 01:03 PM, Eric Botcazou wrote:
> 2012-03-21 Eric Botcazou
>
> * config/ia64/unwind-ia64.c (uw_install_context): Manually save LC
> if it hasn't been previously saved.
Looks ok, given the other ugliness in this macro.
r~
On Fri, Mar 30, 2012 at 8:11 AM, Rainer Orth
wrote:
> Mike Stump writes:
>
>>> Here is the new patch. OK for trunk if there are no regressions on
>>> Linux/ia32 and Linux/x86-64?
>>
>> Too bad you didn't test 32-bit darwin, causes:
>>
>> http://gcc.gnu.org/PR52784
>>
>> Could you please revert
On Fri, Mar 30, 2012 at 8:19 AM, Richard Henderson wrote:
> On 03/30/2012 11:11 AM, Richard Henderson wrote:
>> On 03/30/2012 11:03 AM, Teresa Johnson wrote:
>>> +(define_insn "*movhi_imm_internal"
>>> + [(set (match_operand:HI 0 "memory_operand" "=m")
>>> + (match_operand:HI 1 "immediate_
Hi Richard, Jan and H.J.,
Thanks for all the quick responses and suggestions.
I had tested my patch when tuning for an arch without the LCP stalls,
but it didn't hit an issue in reload because it didn't require
rematerialization. Thanks for pointing out this issue.
Regarding the penalty, it can
Hi,
So here's an extended variant of my hack that implements throwing asms.
Like rth proposed I've added a new pseudo clobber, "throw":
int f (void)
{
int x, y;
x = 1;
y = 2;
try {
__asm__ ("" : "=r"(x), "=r"(y) : : "throw");
} catch (...) {
return 2+x+y;
}
return x+y;
}
On 03/30/2012 12:37 PM, Michael Matz wrote:
> Not yet regstrapped, so no rfa, but does this seem sane?
It definitely seems plausible. I can't immediately think of
anything you might have forgotten.
r~
On 03/30/2012 12:43 PM, Richard Henderson wrote:
> On 03/30/2012 12:37 PM, Michael Matz wrote:
>> Not yet regstrapped, so no rfa, but does this seem sane?
>
> It definitely seems plausible. I can't immediately think of
> anything you might have forgotten.
Stating the obvious, but you'd do well t
On 03/20/2012 05:41 AM, Tristan Gingold wrote:
> 2012-03-20 Tristan Gingold
>
> * ggc-page.c (PAGE_L1_SIZE, PAGE_L2_SIZE, LOOKUP_L1, LOOKUP_L2)
> (ggc_allocated_p, lookup_page_table_entry, set_page_table_entry)
> (alloc_page, init_ggc, clear_marks, struct ggc_pch_data)
>
On 29/03/12 20:34, dann frazier wrote:
> This is an updated version of a patch Debian and Ubuntu are using to
> use an alternate linker path for hardfloat binaries. The difference
> with this one is that it covers the case where no float flag
> was passed in, defaulting to the softfloat path.
>
>
This showed up on i686-linux with BOOT_CFLAGS='-Os -g'. I didn't
investigate the full call chain for why the variable wasn't set
in the by-reference call to fortran_common.
r~
* dwarf2out.c (gen_variable_die): Initialize off.
diff --git a/gcc/dwarf2out.c b/gcc/dwarf2out.c
index 828e996.
On Fri, Mar 30, 2012 at 09:18:13AM -0700, H.J. Lu wrote:
> On Fri, Mar 30, 2012 at 8:11 AM, Rainer Orth
> wrote:
> > Mike Stump writes:
> >
> >>> Here is the new patch. OK for trunk if there are no regressions on
> >>> Linux/ia32 and Linux/x86-64?
> >>
> >> Too bad you didn't test 32-bit darwin,
As discussed in enhancement PR 52607, relying on op0 != op1 can be tricky,
especially when testing_p, and we'd rather not generate oodles of rtl.
Tested on x86_64 and i686-linux and committed.
r~
2012-03-30 Richard Henderson
* config/i386/i386.c (struct expand_vec_perm_d): Add one_op
Losing REG_ARGS_SIZE notes leads to assertion failures in dwarf2cfi.c
when the values don't match across CFG edges.
I'd like to leave this patch on mainline for a week or so to see what
falls out before copying it to the 4.7 branch. There is a new abort
here, though I suppose that would probably
On Fri, Mar 30, 2012 at 11:05 AM, Jack Howarth wrote:
> On Fri, Mar 30, 2012 at 09:18:13AM -0700, H.J. Lu wrote:
>> On Fri, Mar 30, 2012 at 8:11 AM, Rainer Orth
>> wrote:
>> > Mike Stump writes:
>> >
>> >>> Here is the new patch. OK for trunk if there are no regressions on
>> >>> Linux/ia32 and
ramana
i get the same failure on the trunk without my patch.
kenny
On 03/30/2012 07:36 AM, Ramana Radhakrishnan wrote:
Hi
I have tested this on an x86_64 with both the force lowering on and off and
neither cause any regressions as well as extensive testing on my port.
So, just out of curi
On Fri, Mar 30, 2012 at 11:32:37AM -0700, H.J. Lu wrote:
> On Fri, Mar 30, 2012 at 11:05 AM, Jack Howarth
> wrote:
> > On Fri, Mar 30, 2012 at 09:18:13AM -0700, H.J. Lu wrote:
> >> On Fri, Mar 30, 2012 at 8:11 AM, Rainer Orth
> >> wrote:
> >> > Mike Stump writes:
> >> >
> >> >>> Here is the new
On Fri, Mar 30, 2012 at 1:23 PM, Jack Howarth wrote:
> On Fri, Mar 30, 2012 at 11:32:37AM -0700, H.J. Lu wrote:
>> On Fri, Mar 30, 2012 at 11:05 AM, Jack Howarth
>> wrote:
>> > On Fri, Mar 30, 2012 at 09:18:13AM -0700, H.J. Lu wrote:
>> >> On Fri, Mar 30, 2012 at 8:11 AM, Rainer Orth
>> >> wrot
> Hi Richard, Jan and H.J.,
>
> Thanks for all the quick responses and suggestions.
>
> I had tested my patch when tuning for an arch without the LCP stalls,
> but it didn't hit an issue in reload because it didn't require
> rematerialization. Thanks for pointing out this issue.
>
> Regarding th
> > May I apply the patch I posted? It boostrapped/regtested fine on
> > x86-64/Linux.
>
> Yes.
Thanks. Unfortunately, while this was the last identified problem on x86,
another issue is visible on x86-64 as a miscompilation of XML/Ada at -O0.
Reduced testcase attached:
gnat.dg/pack18.adb
gn
In libgo, system calls that return errors convert from an errno value to
an error interface, a step that requires memory allocation. Memory
allocation should be done while the goroutine is running on a thread
that the Go scheduler knows about, which is to say not between calls to
Entersyscall and
I have committed a patch to update libgo to the weekly.2012-03-13
release. As usual this e-mail message only includes the changes to the
files specific to gccgo. Bootstrapped and ran Go testsuite on
x86_64-unknown-linux-gnu. Committed to mainline and 4.7 branch.
Ian
diff -r 2d2ecf6f57c5 libgo/
"Naveen H. S" wrote:
> The patch was tested with movml testcase and works as expected.
>
> Tested with sh2a-elf. No new regressions.
Thanks for testing. I've committed it as revision 186024
on trunk.
Regards,
kaz
"Naveen H. S" wrote:
> Please find attached the patch "crt1.patch" which fixes compilation
> issue with sh2a-single-only target.
> Currently, compilation generates the following error:-
> "merge of architecture 'sh3e' with architecture 'sh2a' produced unknown
> architecture"
> The patch fixes th
I have committed a patch to update libgo to the weekly.2012-03-22
release. As usual, this e-mail only includes the changes to files
specific go gccgo. Bootstrapped and ran Go testsuite on
x86_64-unknown-linux-gnu. Committed to mainline and 4.7 branch.
Ian
diff -r 95129a2d27f4 libgo/MERGE
--- a
I want to revive this patch for mainline and have some questions on
Honza's comments.
On Fri, Apr 29, 2011 at 1:48 PM, Jan Hubicka wrote:
>>
>> A known limitation is that value profiling is not yet sampled, but it
>> does not seem to cause problems.
>
> For the performance alone, we probably don'
On 30 March 2012 20:29, Kenneth Zadeck wrote:
> ramana
>
> i get the same failure on the trunk without my patch.
>
In which case I apologise and will file a bug report separately. I
should really have checked :( .
Ramana
>
> kenny
>
> On 03/30/2012 07:36 AM, Ramana Radhakrishnan wrote:
>>
>> Hi
I have committed a patch to libgo to update to the weekly.2012-03-27
release. This brings libgo up to the Go 1 standard release. This patch
is small enough to include in this e-mail message in its entirety.
Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu.
Committed to mainline and 4
On Fri, Mar 30, 2012 at 5:47 AM, Michael Matz wrote:
> Hi,
>
> On Thu, 29 Mar 2012, Sriraman Tallam wrote:
>
>> +struct __processor_model
>> +{
>> + /* Vendor. */
>> + unsigned int __cpu_is_amd : 1;
>> + unsigned int __cpu_is_intel : 1;
>> + /* CPU type. */
>> + unsigned int __cpu_is_intel_at
Roman, Andrey,
Sorry for the delayed response.
It would indeed be good to have SMS apply to more loop patterns, still
within the realm of *countable* loops. SMS was originally designed to
handle doloops, with a specific pattern controlling the loop, easily
identified and separable from the loop's
Hi,
Hi,
currently all VMS compilers are built on Unix. So, to build the libstdc++
library, this looks like the minimum required.
Tested by build libstdc++ for ia64-hp-openvms.
Ok for trunk ?
I understand that without your patch a libstdc++ for ia64-hp-openvms
cannot be build at all, and I d
Hi again,
On 03/30/2012 12:26 AM, Paolo Carlini wrote:
On 03/29/2012 09:27 PM, Jason Merrill wrote:
On 03/29/2012 03:06 PM, Paolo Carlini wrote:
The exception specification on old_decl doesn't matter; we can drop
that test.
I seem to remember something going wrong with templates otherwise,
be
Work around http://gcc.gnu.org/PR52796 in gcc-4.6 by adding an
overload of each function that passes a parameter pack directly as the
only arguments of an object's constructor, which explicitly takes no
arguments in place of the pack.
Tested with check-c++ and by trying to provoke the bug in valgr
Hi,
tested x86_64-linux, committed mainline and 4_7-branch.
Thanks,
Paolo.
///
2012-03-30 Jeffrey Yasskin
Paolo Carlini
PR libstdc++/52799
* include/bits/deque.tcc (emplace): Fix thinko, replace push_front
-> emplace_front, a
79 matches
Mail list logo
