On 29/05/12 09:31, Jim MacArthur wrote:
2012-05-28 Jim MacArthur
* config/aarch64/t-aarch64: Delete.
* config.host (aarch64*-*-elf): Remove reference to t-aarch64.
I've committed this patch to the aarch64 branch.
/Marcus
On 31/05/12 16:33, Jim MacArthur wrote:
* config/aarch64/sync-cache.S: New file.
Corrected to sync-cache.c and committed to aarch64 branch.
/Marcus
I've merged revision 188326 from the gcc 4.7 branch to the gccgo
branch. I also included the two patches I sent out and have committed
to mainline but have not yet committed to 4.7 branch.
Ian
On 01/06/12 14:29, Jim MacArthur wrote:
In response to a comment from
http://gcc.gnu.org/ml/gcc-patches/2012-05/msg01721.html, this patch removes
t-softfp-sfdf and t-softfp-excl from the aarch64 entries in libgcc/config.host.
Every setting in these files is overridden by t-softfp.
Addition t
On 01/06/12 09:56, Sofiane Naci wrote:
Hi,
This patch re-factors TLS dialect option selection in the AArch64 port to
use the generic support for enumerated option arguments.
Thanks
Sofiane
-
2012-06-01 Sofiane Naci
[AArch64] Use Enums for TLS option selection.
* config/
On 01/06/12 09:53, Sofiane Naci wrote:
Hi,
This patch replaces instances of sprintf with snprintf with sizeof(..) in
the AArch64 port.
It also fixes layout issues in the code it touches.
Thanks
Sofiane
-
ChangeLog
2012-06-01 Sofiane Naci
[AArch64] Replace sprintf with snprintf.
Committed to ARM/embedded-4_6-branch
2012-06-08 Joey Ye
Backport r184442 from mainline
2012-02-21 Richard Earnshaw
PR target/52294
* thumb2.md (thumb2_shiftsi3_short): Split register and
immediate shifts. For register shifts tie operands 0
Hi
In rtl expression, substract a constant c is expressed as add a value -c, so it
is alse processed by adddi3, and I extend it more to handle a subtraction of
64bit constant. I created an insn pattern arm_subdi3_immediate to specifically
represent substraction with 64bit constant while continue k
Richard Henderson writes:
> diff --git a/gcc/testsuite/lib/target-supports.exp
> b/gcc/testsuite/lib/target-supports.exp
> index 1a25324..b93dc5c 100644
> --- a/gcc/testsuite/lib/target-supports.exp
> +++ b/gcc/testsuite/lib/target-supports.exp
> @@ -3795,6 +3795,8 @@ proc check_effective_target
Hi all,
here is a patch for an ICE-on-invalid bug, which concerns the
allocation of CLASS variables. The ICE is fixed by changing the order
of the checks which are done in gfc_match_allocate, so that an error
is triggered before the ICE can occur.
Regtested on x86_64-unknown-linux-gnu. Ok for tru
On 06/06/12 21:45, Gerald Pfeifer wrote:
Looks good to me. (Note, that as someone with write access to the
GCC repository you don't need approval for changes like this, though
I am always happy to look at changes.)
Ok, thank you, I hadn't realized approval was not required.
Change committed.
> [Side note: The piece of code which I'm moving contains a FIXME
> comment, which I don't quite understand, so I'm not sure whether it is
> still valid. It was added by Steve in
> http://gcc.gnu.org/viewcvs?view=revision&revision=145331. Does anyone
> have an opinion on this?]
Possibly the commen
Hi,
Janus Weil wrote:
here is a patch for an ICE-on-invalid bug, which concerns the
allocation of CLASS variables. The ICE is fixed by changing the order
of the checks which are done in gfc_match_allocate, so that an error
is triggered before the ICE can occur.
Regtested on x86_64-unknown-linux
Hi,
>> here is a patch for an ICE-on-invalid bug, which concerns the
>> allocation of CLASS variables. The ICE is fixed by changing the order
>> of the checks which are done in gfc_match_allocate, so that an error
>> is triggered before the ICE can occur.
>>
>> Regtested on x86_64-unknown-linux-gn
Janus Weil wrote:
In any case, for those two, it does not trigger but one
gets later (resolve?) the error:
Error: Allocate-object at (1) must be ALLOCATABLE or a POINTER
Then I guess one can just remove the FIXME.
That's fine with me.
Regarding the error:
Error: Allocate-object at (1) is
Ian Lance Taylor writes:
> @@ -326,13 +336,18 @@
> }
>
> {
> + text="T"
> + case "$GOARCH" in
> + ppc*) text="D" ;;
This is wrong for ppc.
Andreas.
diff --git a/libgo/testsuite/gotest b/libgo/testsuite/gotest
index da1162e..208cbaf 100755
--- a/libgo/testsuite/gotest
+++ b/libg
This fixes most of the libgo testsuite failures.
Andreas.
diff --git a/libgo/Makefile.am b/libgo/Makefile.am
index 1d761bc..7923567 100644
--- a/libgo/Makefile.am
+++ b/libgo/Makefile.am
@@ -1806,6 +1806,7 @@ CHECK = \
LD_LIBRARY_PATH="`${PWD_COMMAND}`/.libs:$${libgccdir}:${LD_LIBRARY_PA
This patch adds a heuristic to the vectorizer when estimating the
minimum profitable number of iterations. The heuristic is
target-dependent, and is currently disabled for all targets except
PowerPC. However, the intent is to make it general enough to be useful
for other targets that want to opt
On Fri, Jun 08, 2012 at 12:40:10PM +0200, Janus Weil wrote:
>
> [Side note: The piece of code which I'm moving contains a FIXME
> comment, which I don't quite understand, so I'm not sure whether it is
> still valid. It was added by Steve in
> http://gcc.gnu.org/viewcvs?view=revision&revision=14533
>> [Side note: The piece of code which I'm moving contains a FIXME
>> comment, which I don't quite understand, so I'm not sure whether it is
>> still valid. It was added by Steve in
>> http://gcc.gnu.org/viewcvs?view=revision&revision=145331. Does anyone
>> have an opinion on this?]
>>
>
> It's bee
> Attached is a third version of the patch, which I will commit soon. It
> adds at least the relevant standard reference to the resolution-stage
> check, too, and fixes the regressions resulting from the changed error
> message.
Committed as r188335.
Cheers,
Janus
On Fri, Jun 08, 2012 at 07:10:02PM +0200, Janus Weil wrote:
> >> [Side note: The piece of code which I'm moving contains a FIXME
> >> comment, which I don't quite understand, so I'm not sure whether it is
> >> still valid. It was added by Steve in
> >> http://gcc.gnu.org/viewcvs?view=revision&revis
Hi, Paolo.
Here is the new version of patch. I have tested this version with gcc
testsuite only on i686 without new regressions, for now. Mips and arm
tests are in progress.
One strange thing I noticed:
>
> No need for this gen_reg_rtx, either, by passing a NULL_RTX target below.
>
>> + carry
Hi!
This patch adds dwz -m DWARF extensions.
See http://www.dwarfstd.org/ShowIssue.php?issue=120604.1&type=open
for more details and
http://sourceware.org/git/?p=dwz.git;a=shortlog;h=refs/heads/multifile
for the tool that creates it. Compared to the DWARF5 proposal, this
is so far implemented as
http://gcc.gnu.org/viewcvs?view=revision&revision=188336
Revision 179760 changed casesi so that the libgcc code to read from
the lookup table by means of __tablejump2__ is used unconditionally,
i.e. does no more depend on -mcall-prologues.
Consequently, casesi costs no more depend on -mcall-prolo
On 06/01/2012 01:58 PM, Sterling Augustine wrote:
It also adds and documents a new option "-g[no-]pubtypes" which allows users
to generate pubtypes even if the target disables them by default.
Hmm, I thought the convention for this sort of flag was to start with
-f, that -g flags were only for
OK.
Jason
On Fri, Jun 08, 2012 at 02:45:09PM -0400, Jason Merrill wrote:
> On 06/01/2012 01:58 PM, Sterling Augustine wrote:
> >It also adds and documents a new option "-g[no-]pubtypes" which allows users
> >to generate pubtypes even if the target disables them by default.
>
> Hmm, I thought the convention
> +/* Extensions for DWZ multifile.
> + See http://www.dwarfstd.org/ShowIssue.php?issue=120604.1&type=open . */
I'd suggest repeating this comment below:
> --- include/dwarf2.h.jj 2012-05-09 20:15:32.0 +0200
> +++ include/dwarf2.h 2012-05-14 12:16:37.414204203 +0200
> @@ -333,6 +333
The following patch is to speed LRA up a bit. It also adds 2012 to
copyright.
The patch was sucessfully bootstrapped on x86-64.
Committed as rev. 188341.
2012-06-08 Vladimir Makarov
* lra-assigns.c: Add 2012 to copyright header. Use lra_assert
instead of gcc_assert.
> Hmm, I thought the convention for this sort of flag was to start with -f,
> that -g flags were only for selecting the type and level of debug info. But
> I see that there are other debug dialect switches that use -g, so I guess
> this is OK.
I kind of prefer -g, but I did notice that it's -fdeb
On Fri, Jun 8, 2012 at 2:22 PM, Cary Coutant wrote:
>> Hmm, I thought the convention for this sort of flag was to start with -f,
>> that -g flags were only for selecting the type and level of debug info. But
>> I see that there are other debug dialect switches that use -g, so I guess
>> this is O
On Tue, Jun 5, 2012 at 7:36 AM, Richard Guenther wrote:
>
> 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
On May 3, 2012, at 9:31 AM, Steven Bosscher wrote:
> How does one turn random volunteer GCC hackers into spoiled brats?
So, brats aside, your new patch breaks my port...
diff --git a/gcc/Makefile.in b/gcc/Makefile.in
index 145e8b8..1eeeab9 100644
--- a/gcc/Makefile.in
+++ b/gcc/Makefile.in
@@ -35
The following patch fixing some LRA crashes (when I tried a new
heuristics in insn alternative selection) on GCC tests on ppc.
The patch was successfully bootstrapped on x86-64 and power7.
Committed as rev. 188350.
2012-06-08 Vladimir Makarov
* emit-rtl.c (validate_subreg): Don't ch
On Apr 17, 2012, at 2:08 PM, Bernd Schmidt wrote:
> This patch enables GET_MODE_WIDER_MODE for MODE_PARTIAL_INT (by setting
> the wider mode to the one the partial mode is based on), which is useful
> for the port I'm working on: I can avoid defining operations on the
> partial modes.
I think this
See http://gcc.gnu.org/ml/gcc-patches/2012-01/msg01427.html for the
previous discussion about the motivation for this patch and the choice
of __pthread_key_create (also the comment added in the code by this patch).
This had no effect on testsuite results on x86_64-linux-gnu
(Ubuntu EGLIBC 2.11.1-
On Jun 8, 2012, at 4:11 PM, Mike Stump wrote:
> On Apr 17, 2012, at 2:08 PM, Bernd Schmidt wrote:
>> This patch enables GET_MODE_WIDER_MODE for MODE_PARTIAL_INT (by setting
>> the wider mode to the one the partial mode is based on), which is useful
>> for the port I'm working on: I can avoid defini
Hi,
This patch fixes the failure test case that I had submitted the last time.
The changeLog is testsuite/gChangeLog.gimplefe is as follows.
2012-06-09 Sandeep Soni
* gimple.dg/20120605-2.gimple : New.
While the changelog in gimple/ChangeLog is as follows
2012-06-09 Sandeep S
On Tue, 5 Jun 2012, Sandra Loosemore wrote:
> (1) While the address cost computation is assuming in some situations
> that pre/post increment/decrement addressing will be used if
> supported by the target, it isn't actually using the target's address
> cost for such forms -- instead, just the cost
On Tue, 5 Jun 2012, Steven Bosscher wrote:
> 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?
Woohoo, that's a lot of left-over cruft!
@@
41 matches
Mail list logo
