On Mon, Jun 30, 2014 at 9:59 PM, Uros Bizjak ubiz...@gmail.com wrote:
After fixing _.barriers and _.eh_range passes w.r.t. CALL_ARG_LOCATION
notes, we can finaly move handling of trap shadows (PR 56858) and insn
alignments into their own passes.
Additionally, the patch skips handling of
Hi,
Thank you for your comments.
...stdbar should never be generated since #StoreStore is implied by
TSO.
I missed that #StoreStore is never generated for TSO, so I'm removing
that pattern.
OK, thanks. Does this result in a significance performance gain?
stb seems to be at least twice
ChangeLog:
2014-07-11 Daniel Cederman ceder...@gaisler.com
gcc/config/sparc/
* sync.md: Generate more efficient memory barriers for LEON3
---
gcc/config/sparc/sync.md | 21 +++--
1 file changed, 19 insertions(+), 2 deletions(-)
diff --git a/gcc/config/sparc/sync.md
On Thu, 10 Jul 2014, Jakub Jelinek wrote:
On Thu, Jul 10, 2014 at 05:12:53PM +0200, Richard Biener wrote:
The following makes sure we fold the reads from string constants
created from folding memcpy (foo, string_cst[0], ...) to
eventually create similar code as if that memcpy folding
On Thu, 10 Jul 2014, Jakub Jelinek wrote:
On Thu, Jul 10, 2014 at 04:30:13PM +0200, Richard Biener wrote:
Compromise hack below. It simply avoids the transform for
sources that c_strlen can compute a length of. That fixes all
strlenopt testcase apart from strlenopt-8.c which does
On Mon, May 5, 2014 at 7:17 PM, Rong Xu x...@google.com wrote:
Here is the updated patch. The difference with patch set 3 is
(1) use the gcov-counter.def for scaling operation.
(2) fix wrong scaling function for time-profiler.
passed with bootstrap, profiledboostrap and SPEC2006.
One of the
On Thu, Jul 10, 2014 at 8:22 PM, Jeff Law l...@redhat.com wrote:
On 07/10/14 09:48, Michael Matz wrote:
Hi,
On Thu, 10 Jul 2014, Richard Biener wrote:
Apart from the out-of-SSA patch you proposed elsewhere a possibility
is to simply never mark undefined SSA names as
On Fri, Jul 11, 2014 at 10:00:01AM +0200, Richard Biener wrote:
I started with adding an optional arg to native_encode_expr but then
figured that all the current workers return failure if the entire
expr doesn't fit within 'len'. But what I want is a native_encode_expr
That is because the
On 11 July 2014 10:07, Richard Biener richard.guent...@gmail.com wrote:
On Mon, May 5, 2014 at 7:17 PM, Rong Xu x...@google.com wrote:
Here is the updated patch. The difference with patch set 3 is
(1) use the gcov-counter.def for scaling operation.
(2) fix wrong scaling function for
On Fri, Jul 11, 2014 at 10:10 AM, Richard Biener
richard.guent...@gmail.com wrote:
On Thu, Jul 10, 2014 at 8:22 PM, Jeff Law l...@redhat.com wrote:
On 07/10/14 09:48, Michael Matz wrote:
Hi,
On Thu, 10 Jul 2014, Richard Biener wrote:
Apart from the out-of-SSA patch you proposed elsewhere a
On Fri, 11 Jul 2014, Jakub Jelinek wrote:
On Fri, Jul 11, 2014 at 10:00:01AM +0200, Richard Biener wrote:
I started with adding an optional arg to native_encode_expr but then
figured that all the current workers return failure if the entire
expr doesn't fit within 'len'. But what I want
Are we sure it ever stabilizes? But yes, I had something like this in mind
(just do one iteration always) in case we need to improve hashing.
Hi,
this is bit quick experiment with strengthening the hash by iteration. I don't
seem to be able to measure WPA time difference for the patch though
Hello,
I fixed Options That Control Optimization section according to 'gcc -Q
--help=optimizers' and after consultation with Jakub, I added missing
-foptimize-strlen option.
Ready for trunk?
Martin
ChangeLog:
2014-07-11 Martin Liska mli...@suse.cz
* doc/invoke.texi: Added missing
I would use -x to see the headers. This should tell you if it is defined
or only used there.
It gives the following output:
roman@roman-System-Product-Name:~/isl-0.12.2/lib$ objdump -x
libisl.so.10.2.2 | grep isl_ast_expr_get_val
00060380 g F .text 0053
PSImode is 20 bits, fits in a 20 bit register, and uses 20 bit operations.
Then why do you need this change?
- TYPE_SIZE (type) = bitsize_int (GET_MODE_BITSIZE (TYPE_MODE (type)));
+ TYPE_SIZE (type) = bitsize_int (GET_MODE_PRECISION (TYPE_MODE
(type))); TYPE_SIZE_UNIT (type) =
Thank to you all.
Committed revision 212450.
--Marat
Original Message
Subject:Re: [PING][PATCH] Fix for PR 61561
Date: Thu, 10 Jul 2014 14:01:24 +0100
From: Ramana Radhakrishnan ramana.radhakrish...@arm.com
To: Marat Zakirov m.zaki...@samsung.com,
On Fri, 11 Jul 2014, Jan Hubicka wrote:
Are we sure it ever stabilizes? But yes, I had something like this in mind
(just do one iteration always) in case we need to improve hashing.
Hi,
this is bit quick experiment with strengthening the hash by iteration. I don't
seem to be able to
Hi Cesar!
On Thu, 10 Jul 2014 11:43:11 -0700, Cesar Philippidis ce...@codesourcery.com
wrote:
This patch addresses two bugs openacc reduction clause bugs.
Thanks! OK; one question/suggestion, though:
--- a/gcc/omp-low.c
+++ b/gcc/omp-low.c
@@ -9679,11 +9679,23 @@ process_reduction_data
On Fri, 11 Jul 2014, Richard Biener wrote:
On Fri, 11 Jul 2014, Jakub Jelinek wrote:
On Fri, Jul 11, 2014 at 10:00:01AM +0200, Richard Biener wrote:
I started with adding an optional arg to native_encode_expr but then
figured that all the current workers return failure if the entire
The full barrier pattern membar_leon3 also gets generated so I think
that one should be kept also.
Do you have a testcase? membar is generated by sparc_emit_membar_for_model
and, for the TSO model of LEON3, implied = StoreStore | LoadLoad | LoadStore
so mm can only be StoreLoad, which means
Hi,
since we both agreed offlining constructors from global decl stream is a good
idea, I went ahead and implemented it. I would like to followup by an
cleanups; for example the sections are still tagged as function sections, but I
would like to do it incrementally. There is quite some uglyness
I tried to incorporate all your comments in the attached patch.
Nice. It looks now very good to me. One final and last detail:
+/* TREE_FROM_ISL_ID maps ISL's scattering and parameter identifiers
+ to corresponding trees. */
+
+typedef struct ivs_params {
+ std::mapisl_id *, tree
Hi Cesar!
On Thu, 10 Jul 2014 11:47:42 -0700, Cesar Philippidis ce...@codesourcery.com
wrote:
These patch enables the collapse clause with a value greater than one.
Thanks!
Is this patch OK for gomp-4_0-branch?
OK with the following addressed:
--- a/gcc/c/c-parser.c
+++
Well - as we re-use the streamer cache to store the hash value it isn't
really possible to do better ... (at least I don't have a clever idea)
OK, no cleverness on my side either.
Yeah (though you wouldn't need the extra hashing - we only need to know
the hash of the SCC). The current
On Fri, Jul 11, 2014 at 11:13:12AM +0200, Richard Biener wrote:
+ if (offset = off)
+ ptr[offset - off] = value;
For original off != 0, you aren't checking whether offset - off len
though (various places), you don't want to write beyond end of buffer.
- return total_bytes;
+
On Fri, 11 Jul 2014, Jan Hubicka wrote:
Well - as we re-use the streamer cache to store the hash value it isn't
really possible to do better ... (at least I don't have a clever idea)
OK, no cleverness on my side either.
Yeah (though you wouldn't need the extra hashing - we only
Hi,
this first patch continues with rafactoring of IPA infrastructure so that
we will have C++ API. In the patch, I transformed many global functions to
members of symtab_node and cgraph_node.
Example:
cgraph_remove_node (struct cgraph_node *node) - cgraph_node::remove (void)
We hash only on outgoing SCC edges. You can easily have
main variant type T and variants T1,T2 that are same all used by type
T again.
Ok, but then the two variants shouldn't be in the same SCC or at
least not in the same SCC as the main variant.
They will because T refers them
On Fri, 11 Jul 2014, Jakub Jelinek wrote:
On Fri, Jul 11, 2014 at 11:13:12AM +0200, Richard Biener wrote:
+ if (offset = off)
+ ptr[offset - off] = value;
For original off != 0, you aren't checking whether offset - off len
though (various places), you don't want to write
Hi!
On Thu, 22 May 2014 11:31:25 +0400, Ilmir Usmanov i.usma...@samsung.com wrote:
On 16.05.2014 19:44, Ilmir Usmanov wrote:
On 16.05.2014 19:12, Thomas Schwinge wrote:
You recently indicated that you have already begun implementing OpenACC
subarray specifications in the GCC Fortran front
On Fri, 11 Jul 2014, Jan Hubicka wrote:
We hash only on outgoing SCC edges. You can easily have
main variant type T and variants T1,T2 that are same all used by type
T again.
Ok, but then the two variants shouldn't be in the same SCC or at
least not in the same SCC as the
Hi,
only today it occurred to me that we can as well delay all the
diagnostic at issue to the check_narrowing at the end of
convert_like_real and avoid at once possible issues with duplicate
warnings. Tested x86_64-linux.
Thanks,
Paolo.
//
/cp
2014-07-11 Paolo Carlini
That was an error on my side. The wrong memory model had gotten cached
in a generated make script. Lets drop membar_leon3 also then :)
On 2014-07-11 11:15, Eric Botcazou wrote:
The full barrier pattern membar_leon3 also gets generated so I think
that one should be kept also.
Do you have a
ChangeLog:
2014-07-11 Daniel Cederman ceder...@gaisler.com
gcc/config/sparc/
* sync.md: Generate more efficient memory barriers for LEON3
---
gcc/config/sparc/sync.md | 10 +-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/gcc/config/sparc/sync.md
On 10/07/14 08:46, Terry Guo wrote:
Hi there,
Currently the insn type of DSP-kind instructions like QSUB8 is alu_reg which
is same as other normal instructions like SUB. In order to distinguish those
DSP-kind instructions, this patch intends to replace current alu_reg with
two sub
On 10/07/14 09:06, Terry Guo wrote:
Hi there,
As the second and final patch in this series, it intends to update alu_reg
and alus_reg types for AArch64 port. With this change, the gcc can be
successfully built for AArch64. Is it OK to trunk?
BR,
Terry
2014-07-10 Terry Guo
On Fri, Jul 11, 2014 at 12:11:10PM +0200, Thomas Schwinge wrote:
To avoid duplication of work: with Jakub's Fortran OpenMP 4 target
changes recently committed to trunk, and now merged into gomp-4_0-branch,
I have trimmed down Ilmir's patch to just the OpenACC bits, OpenMP 4
target changes
On 10/07/14 11:12, Marcus Shawcroft wrote:
On 1 July 2014 11:05, Christophe Lyon christophe.l...@linaro.org wrote:
* documentation (README)
* dejanu driver (neon-intrinsics.exp)
* support macros (arm-neon-ref.h, compute-ref-data.h)
* Tests for 3 intrinsics: vaba, vld1, vshl
Hi, The
The struct now contains only a single element such that there seems to be no
need for it anymore. Should we remove it? (We could still use a typedef if
you believe the datatype is too long).
I don't mind removing it. However I think that we should choose the
way of transferring of sese region,
On 11/07/2014 13:11, Roman Gareev wrote:
The struct now contains only a single element such that there seems to be no
need for it anymore. Should we remove it? (We could still use a typedef if
you believe the datatype is too long).
I don't mind removing it. However I think that we should
On Fri, 11 Jul 2014, Jan Hubicka wrote:
Hi,
since we both agreed offlining constructors from global decl stream is a good
idea, I went ahead and implemented it. I would like to followup by an
cleanups; for example the sections are still tagged as function sections, but
I
would like to do
Thanks foe the review and suggestions.
On 10/07/14 22:15, Richard Biener wrote:
On Mon, Jul 7, 2014 at 8:55 AM, Kugan kugan.vivekanandara...@linaro.org
wrote:
[...]
For -fwrapv, it is due to how PROMOTE_MODE is defined in arm back-end.
In the test-case, a function (which has signed char
On Fri, 11 Jul 2014, Jan Hubicka wrote:
Hi,
since we both agreed offlining constructors from global decl stream is a
good
idea, I went ahead and implemented it. I would like to followup by an
cleanups; for example the sections are still tagged as function sections,
but I
would
Hmm, walking everything first and then starting streaming sounds bad idea
for locality. Can't I just re-do the walk since I know what the SCC is?
I will read the code more after lunch.
Might be possible with a 2nd SCC stack set up, yes.
I set up hash_map to store the hash values anyway.
On Fri, 11 Jul 2014, Jan Hubicka wrote:
On Fri, 11 Jul 2014, Jan Hubicka wrote:
Hi,
since we both agreed offlining constructors from global decl stream is a
good
idea, I went ahead and implemented it. I would like to followup by an
cleanups; for example the sections are
On Fri, 11 Jul 2014, Jan Hubicka wrote:
Hmm, walking everything first and then starting streaming sounds bad idea
for locality. Can't I just re-do the walk since I know what the SCC is?
I will read the code more after lunch.
Might be possible with a 2nd SCC stack set up, yes.
I
On Fri, 11 Jul 2014, Jan Hubicka wrote:
Hmm, walking everything first and then starting streaming sounds bad
idea
for locality. Can't I just re-do the walk since I know what the SCC is?
I will read the code more after lunch.
Might be possible with a 2nd SCC stack set
Hi,
On Thu, 10 Jul 2014, Jeff Law wrote:
The insight to note is, that undefined SSA names should really be
coalesced with something (otherwise you lost an optimization opportunity),
but it doesn't matter with _what_ each use of the undefined name is
coalesced, you can even identify
On Fri, 11 Jul 2014, Jan Hubicka wrote:
On Fri, 11 Jul 2014, Jan Hubicka wrote:
Hmm, walking everything first and then starting streaming sounds bad
idea
for locality. Can't I just re-do the walk since I know what the SCC
is?
I will read the code more after lunch.
Well, just make them regular (anonymous) VAR_DECLs then ... (the fact
that a CONST_DECL is anonymous is probably the only real difference -
and that they are mergeable by content).
Something like that, perhaps. Plan to do that incrementally - having them in
symbol tabel first is an
Ah, ok. Well, let's hope walk_tree walks all edges the DFS walk walks ;)
A quick look tells me it doesn't walk DECL_VINDEX or
DECL_FUNCTION_PERSONALITY for example.
I guess it should - will try patch adding that :) Both seems like just
ommisions - these fileds were added quite recently
Hi,
our kernel guys saw this problem when they used an asm goto to implement a
conditional where the then-block was empty. RTL ifcvt happily goes on and
removes the whole block and the jump instruction, even though it has
side-effects (as marked with the volatility). Patch below fixes it by
On 10/07/14 22:53, Maxim Kuvyrkov wrote:
On Jul 10, 2014, at 8:00 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote:
On 30/06/14 21:39, Jeff Law wrote:
On 06/27/14 02:29, Kyrill Tkachov wrote:
Hi all,
This patch generalises the TARGET_MACRO_FUSION_PAIR_P hook usage to work
on more than just
On Fri, Jul 11, 2014 at 1:52 PM, Kugan
kugan.vivekanandara...@linaro.org wrote:
Thanks foe the review and suggestions.
On 10/07/14 22:15, Richard Biener wrote:
On Mon, Jul 7, 2014 at 8:55 AM, Kugan kugan.vivekanandara...@linaro.org
wrote:
[...]
For -fwrapv, it is due to how PROMOTE_MODE
I noticed that we set node-definition = true in varpool_assemble_decl.
The surrounding code suggests that we should only ever get there if
definition is already true, so I changed it to an assert. The question
is interesting for some modifications I'm making for ptx (which requires
On Fri, 2014-07-11 05:48:08 -, x...@gcc.gnu.org x...@gcc.gnu.org wrote:
Author: xur
Date: Fri Jul 11 05:48:07 2014
New Revision: 212448
URL: https://gcc.gnu.org/viewcvs?rev=212448root=gccview=rev
Log:
2014-07-10 Rong Xu x...@google.com
Add gcov-tool: an offline gcda profile
On Fri, 2014-07-11 15:03:06 +0200, Jan-Benedict Glaw jbg...@lug-owl.de wrote:
See eg. http://toolchain.lug-owl.de/buildbot/show_build_details.php?id=289639,
it breaks like this:
[...]
g++ -c -g -O2 -DIN_GCC -DCROSS_DIRECTORY_STRUCTURE -fno-exceptions
-fno-rtti
Hi Tom,
On 8 July 2014 20:45, Tom de Vries tom_devr...@mentor.com wrote:
On 01-07-14 19:26, Jeff Law wrote:
On 07/01/14 09:51, Yufeng Zhang wrote:
Hi,
This patch resolves a conflict between the aapcs64 test framework for
func-ret tests and the optimization option -fuse-caller-save, which
On Fri, 11 Jul 2014, Kyrill Tkachov wrote:
On 10/07/14 22:53, Maxim Kuvyrkov wrote:
The patch looks good to me, but some cleanup suggestions below.
Thanks, here's an updated patch.
How's this?
You need to remove 'if (targetm. ...) SCHED_GROUP_P (insn) = 1;' from the
first if branch,
I noticed that we set node-definition = true in
varpool_assemble_decl. The surrounding code suggests that we should
only ever get there if definition is already true, so I changed it
to an assert. The question is interesting for some modifications I'm
making for ptx (which requires
On 11/07/14 14:20, Alexander Monakov wrote:
On Fri, 11 Jul 2014, Kyrill Tkachov wrote:
On 10/07/14 22:53, Maxim Kuvyrkov wrote:
The patch looks good to me, but some cleanup suggestions below.
Thanks, here's an updated patch.
How's this?
You need to remove 'if (targetm. ...) SCHED_GROUP_P
On Fri, 11 Jul 2014, Richard Biener wrote:
On Thu, 10 Jul 2014, Jakub Jelinek wrote:
On Thu, Jul 10, 2014 at 04:30:13PM +0200, Richard Biener wrote:
Compromise hack below. It simply avoids the transform for
sources that c_strlen can compute a length of. That fixes all
strlenopt
I've attached an improved version of the patch and the ChangeLog
entry. Are they fine for trunk?
--
Cheers, Roman Gareev.
2014-07-11 Roman Gareev gareevro...@gmail.com
gcc/
* graphite-isl-ast-to-gimple.c (gmp_cst_to_tree):
New
On Fri, Jul 11, 2014 at 03:36:15PM +0200, Richard Biener wrote:
*** c_strlen (tree src, int only_value)
*** 606,612
/* If the offset is known to be out of bounds, warn, and call strlen at
runtime. */
! if (offset 0 || offset max)
{
/*
Ping. Added small changes due to previous discussion in community.
Original Message
Subject:[PATCH][Ping v3] Add patch for debugging compiler ICEs
Date: Fri, 04 Jul 2014 18:32:44 +0400
From: Maxim Ostapenko m.ostape...@partner.samsung.com
To: GCC Patches
On 11/07/2014 15:41, Roman Gareev wrote:
I've attached an improved version of the patch and the ChangeLog
entry. Are they fine for trunk?
LGTM.
Thank you!
Tobias
On 10/07/14 22:48 -0400, Samuel Bronson wrote:
PR libstdc++/58962
* python/libstdcxx/v6/printers.py: Port to Python 2+3
(imap): New compat function.
(izip): Likewise.
(Iterator): New mixin to allow writing iterators in Python 3 style
Here is a preprocessor patch to make error messages show C++11
and other relevant C++ language instead of C99.
Built and tested on x86_64-linux.
This caused
FAIL: gcc.dg/cpp/macsyntx.c (test for excess errors)
FAIL: gcc.dg/cpp/macsyntx.c (test for excess errors)
FAIL: gcc.dg/cpp/macsyntx.c
Sorry. This code meant to work with the different mkdir api in
windows. I used wrong ifdef.
Here is the patch. OK for checkin?
Thanks,
-Rong
2014-07-11 Rong Xu x...@google.com
* gcov-tool.c (gcov_output_files): Fix build error.
Index: gcov-tool.c
On Fri, Jul 11, 2014 at 2:34 PM, Michael Matz wrote:
PR rtl-optimization/61772
* ifcvt.c (dead_or_predicable): Check jump to be free of side
effects.
This is OK.
Ciao!
Steven
Tested on x86_64-suse-linux and installed as obvious.
Andreas.
PR preprocessor/61389
* gcc.dg/cpp/macsyntx.c: Update expected warnings.
* gcc.dg/cpp/sysmac1.c: Likewise.
diff --git a/gcc/testsuite/gcc.dg/cpp/macsyntx.c
b/gcc/testsuite/gcc.dg/cpp/macsyntx.c
index
On 19/06/14 21:19, Yuri Gribov wrote:
Thirdly, we also need to fix movhi_bytes (for pre-v4) thumb2_movhi_insn
(for thumb2) and, quite possibly, thumb1_movhi_insn (for thumb1). There
may well be additional changes for movqi variants as well.
A general question: how should one test ARM
This patch enables COMDAT functionality on AIX. It defines
MAKE_DECL_ONE_ONLY and removes the historical legacy link-line option
of -bnodelcsect.
-bnodelcsect instructs the AIX (garbage collecting) linker to preserve
un-referenced CSECTs (like ELF sections). In the distant past, GCC
emitted code
Sorry. This code meant to work with the different mkdir api in
windows. I used wrong ifdef.
Here is the patch. OK for checkin?
OK. I also see the following with LTO bootstrap:
../../gcc/../libgcc/libgcov-util.c:41:24: error: type of �gcov_max_filename�
does not match original declaration
Fix a couple of issues causing a test regression and boostrap build errors.
2014-07-11 Andrew Sutton andrew.n.sut...@gmail.com
* gcc/cp/typeck.c (cp_build_function_call_vec): Emit diagnostic
at the input location.
* gcc/cp/error.c (dump_template_decl): Constraints are
On 10/07/14 22:48 -0400, Samuel Bronson wrote:
When I try to build test just libstdc++, or to run the testsuite
from trunk against my installed libstdc++, the testsuite tries to pass
-fdiagnostics-color=never to the system GCC, which is too old to
know what that is.
Since I really just want
On 10/07/14 22:48 -0400, Samuel Bronson wrote:
We load our pretty-printers explicitly, and we shouldn't need any other
random -gdb.gdb or -gdb.py files from anywhere, so in this patch we turn
that off by running set auto-load no.
Also, run info share so that the list of loaded libraries ends
On 07/10/2014 06:16 AM, Paolo Carlini wrote:
.. I have another comment: are we sure the usual strategy:
templatetypename _UniformRandomNumberGenerator
result_type
operator()(_UniformRandomNumberGenerator __urng)
{ return this-operator()(__urng, this-_M_param); }
doesn't make
I wonder why. The struct definition for gcov_fn_info has not changed
in this patch.
David
On Fri, Jul 11, 2014 at 1:07 AM, Richard Biener
richard.guent...@gmail.com wrote:
On Mon, May 5, 2014 at 7:17 PM, Rong Xu x...@google.com wrote:
Here is the updated patch. The difference with patch set 3
Hi,
On 07/11/2014 05:38 PM, Ed Smith-Rowland wrote:
OK?
Ok, thanks, but please adjust the dates you have on the testcases to the
date of the actual commit (I suppose today or tomorrow)
Thanks again!
Paolo.
What is the macro to test POSIX IO on host? The guard uses
TARGET_POSIX_IO which is not right.
David
On Fri, Jul 11, 2014 at 1:12 AM, Christophe Lyon
christophe.l...@linaro.org wrote:
On 11 July 2014 10:07, Richard Biener richard.guent...@gmail.com wrote:
On Mon, May 5, 2014 at 7:17 PM, Rong
On Fri, Jul 11, 2014 at 08:42:27AM -0700, Xinliang David Li wrote:
I wonder why. The struct definition for gcov_fn_info has not changed
in this patch.
Perhaps it has been used only in C until now?
Jakub
right.
Rong, the fix would be just change ctr array size to 1. For each
function, there should be at least one kind of counters -- see the
assertion in build_fn_info_type. There are some code that do
'sizeof(gcov_fn_info)' when computing heap size -- they can be
adjusted or leave it as it is (if
2014-07-11 12:50 GMT+04:00 S, Pitchumani pitchuman...@atmel.com:
This patch adds supports for Atmel's ata5782 and ata5831 devices.
These devices ISA matches with AVR5 architecture except that text section
start for these two devices is 0x8000. By default 0x0 is used as the text
start address.
PSImode is 20 bits, fits in a 20 bit register, and uses 20 bit operations.
Then why do you need this change?
Because parts of the gcc code use the byte size instead of the bit
size, or round up, or assume powers-of-two sizes.
- TYPE_SIZE (type) = bitsize_int (GET_MODE_BITSIZE
On Fri, Jul 11, 2014 at 8:06 AM, Jan Hubicka hubi...@ucw.cz wrote:
Sorry. This code meant to work with the different mkdir api in
windows. I used wrong ifdef.
Here is the patch. OK for checkin?
OK. I also see the following with LTO bootstrap:
../../gcc/../libgcc/libgcov-util.c:41:24: error:
I should use _WIN32 macro. This code is for windows mkdir api.
I'll commit a patch for this shortly (approved by honza).
-Rong
On Fri, Jul 11, 2014 at 8:49 AM, Xinliang David Li davi...@google.com wrote:
What is the macro to test POSIX IO on host? The guard uses
TARGET_POSIX_IO which is not
I did see the warning in the bootstrap, but it did not exit the build.
I thought it was ok.
I'll have a patch for this and send for review.
-Rong
On Fri, Jul 11, 2014 at 9:13 AM, Xinliang David Li davi...@google.com wrote:
right.
Rong, the fix would be just change ctr array size to 1. For
Hello,
I've been looking through tree-ssa-loop-im.c (while hunting down a bug
in the modula-2 front end) and found a curiosity in
gcc/tree-ssa-loop-im.c.
It seems that there is dead code in function determine_max_movement
as mem_ref_in_stmt can never return NULL.
static mem_ref_p
On 07/11/14 06:05, Michael Matz wrote:
Hi,
On Thu, 10 Jul 2014, Jeff Law wrote:
The insight to note is, that undefined SSA names should really be
coalesced with something (otherwise you lost an optimization opportunity),
but it doesn't matter with _what_ each use of the undefined name is
The following patch fixes a bug in LRA inheritance found on compiling a
big file on s390x. Unfortunately, the bug is very hard to reproduce and
the test is too big and can not be included. But even if it were
included, checking a correct code generation would be a problem too.
The bug was
Gaius Mulley gaius.mul...@southwales.ac.uk writes:
Hello,
I've been looking through tree-ssa-loop-im.c (while hunting down a bug
in the modula-2 front end) and found a curiosity in
gcc/tree-ssa-loop-im.c.
It seems that there is dead code in function determine_max_movement
as
Hi,
Firefox build ICEs in ipa-devirt (types_same_for_odr) not being able
to establish ODR equivalency for non-polymorphic types. This is because
ipa-prop and ipa-cp does call get_binfo_at_offset where it really wants
to propagate on types and offsets. Before this is fixed this patch avoids
the
Ping.
On Thu, Jun 26, 2014 at 10:54 AM, Sriraman Tallam tmsri...@google.com wrote:
Hi Uros,
Could you please review this patch?
Thanks
Sri
On Fri, Jun 20, 2014 at 5:17 PM, Sriraman Tallam tmsri...@google.com wrote:
Patch Updated.
Sri
On Mon, Jun 9, 2014 at 3:55 PM, Sriraman Tallam
Hi,
this is the variant of patch I comitted. I noticed that partitioning actually
calls ctor_for_folding just to figure out if the constant value may be used
that drags in every readonly variable ctor into memory at WPA.
So now we have separate predicate varpool_ctor_useable_for_folding_p to check
On 07/10/2014 06:38 PM, Paolo Carlini wrote:
if (SCALAR_TYPE_P (type))
+ if (!CLASS_TYPE_P (TREE_TYPE (init)))
Why this change? check_narrowing only deals with scalar types.
Jason
Oops, this patch didn't thread with the earlier one...
On 07/11/2014 06:15 AM, Paolo Carlini wrote:
if (SCALAR_TYPE_P (type))
Is the condition still necessary?
- check_narrowing (type, init);
+ flags |= LOOKUP_NO_NARROWING;
Jason
Hi,
On 07/11/2014 07:58 PM, Jason Merrill wrote:
Oops, this patch didn't thread with the earlier one...
Yeah, sorry, I considered my drafts of yesterday evening a distraction
and wanted to start a fresh thread with a complete (more meaningful) new
proposal. Probably not a good idea, after
Richard,
I looked at my patch again. I already add -Wno-error to libgcov-util.o
compilation:
In line 200 of gcc/Makefile.in
libgcov-util.o-warn = -Wno-error
In my test, I used gcc-4.6 as the host compiler. I got warning like this:
In file included from
Hi
this is patch i am going to commit after testing. It removes DECL_INIT_IO
timevar that guards only one variable set (so hardly measure anything) and
moves GIMPLE_IN to proper place. It also adds CTORS_IN and CTORS_OUT.
I get:
ipa lto gimple out : 0.37 ( 0%) usr 0.21 ( 3%) sys 0.64
1 - 100 of 113 matches
Mail list logo
