+euge...@google.com
Hi Christophe,
On Thu, Mar 28, 2013 at 2:09 AM, Christophe Lyon
wrote:
> Hi,
> This small patch enables libsanitizer on ARM.
> It has been tested successfully on cortex-a9 hardware (via the GCC testsuite).
>
> I have chosen to bundle -funwind-table with -fsanitize=* so that a
On Thu, Mar 28, 2013 at 11:36 AM, Konstantin Serebryany
wrote:
> +euge...@google.com
>
> Hi Christophe,
>
> On Thu, Mar 28, 2013 at 2:09 AM, Christophe Lyon
> wrote:
>> Hi,
>> This small patch enables libsanitizer on ARM.
>> It has been tested successfully on cortex-a9 hardware (via the GCC
>> t
On Thu, Mar 28, 2013 at 12:00:23PM +0400, Evgeniy Stepanov wrote:
> We do it because newer versions of Android use PIE binaries, and,
> combined with other specifics of address space on Linux/ARM, there is
> no space for ASan shadow anywhere else. And it's faster.
>
> Zero-based shadow requires PI
On Thu, Mar 28, 2013 at 12:07 PM, Jakub Jelinek wrote:
> On Thu, Mar 28, 2013 at 12:00:23PM +0400, Evgeniy Stepanov wrote:
>> We do it because newer versions of Android use PIE binaries, and,
>> combined with other specifics of address space on Linux/ARM, there is
>> no space for ASan shadow anywh
> Isn't pass_dwarf2_frame enabled whenever we're generating any unwind info?
OK, I've installed this on mainline, 4.8 branch and 4.7 branch.
2013-03-28 Eric Botcazou
* toplev.c (process_options): Do not disable -fomit-frame-pointer on a
general basis if unwind info is request
Foone Turing wrote:
This page: http://gcc.gnu.org/gcc-4.8/
under "release history" says GCC 4.8 was released on March 22, 2012.
This should be 2013, not 2012.
Thanks for the report! I have corrected it now.
Tobias
Index: index.html
==
On Wed, 27 Mar 2013, Marek Polacek wrote:
> This fixes PR56695 where we ICEd because expand_vec_cond_expr
> doesn't expect that the result of a vector comparison is of
> TYPE_UNSIGNED type. So we unconditionally build a signed type now.
> We know that the result of vec comparison is e.g. { -1, 0,
On Wed, Mar 27, 2013 at 5:15 PM, Lawrence Crowl wrote:
> On 3/27/13, Richard Biener wrote:
>> On Mar 27, 2013, Lawrence Crowl wrote:
>>> Patch with rename to debug attached.
>>> Tested on x86_64.
>>>
>>>
>>> Add uniform debug dump function names.
>>>
>>>
>>> Add some overloaded functions that pr
On Wed, Mar 27, 2013 at 6:22 PM, Teresa Johnson wrote:
> I found that the node weight updates on cloned nodes during ipa-cp were
> leading to incorrect/insane weights. Both the original and new node weight
> computations used truncating divides, leading to a loss of total node weight.
> I have fix
On Thu, Mar 28, 2013 at 12:16 AM, Steven Bosscher wrote:
> *Ping*
Ok.
Thanks,
Richard.
> On Sat, Mar 23, 2013 at 5:15 PM, Steven Bosscher wrote:
>> Hello,
>>
>> This patch replaces all tests I could find, where the code is looking
>> for jump table data with GET_CODE(..)=ADDR_VEC and similar wi
On Wed, Mar 27, 2013 at 5:44 PM, Lawrence Crowl wrote:
> On 3/27/13, Richard Biener wrote:
>> On Mar 23, 2013 Lawrence Crowl wrote:
>> > This patch is a consolodation of the hash_table patches to the
>> > cxx-conversion branch.
>> >
>> > Update various hash tables from htab_t to hash_table.
>> >
libgfortran parses the format string for formatted I/O and saves it in
an internal representation. To speed up the I/O - for instance in a loop
-, caching is used.
However, a pointer to format string is used for the processing of
strings (normal string constants and Hollerith). That works well
Tobias Burnus wrote:
b) To copy the format string
The attached patch does the latter. The current hashing algorithm
avoids hash collisions by checking whether the value is exactly the
same - and the value is given by the format string. Thus, instead of
copying the string when storing the form
Richard Biener wrote:
This fixes PR56694 - the code keeping BLOCKs live is not looking
at the EH tree for references. In the must-not-throw failure_loc
such references can now appear.
Fixed by reverting that to 4.7 behavior.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk
On Thu, 28 Mar 2013, Tejas Belagod wrote:
> Richard Biener wrote:
> > This fixes PR56694 - the code keeping BLOCKs live is not looking
> > at the EH tree for references. In the must-not-throw failure_loc
> > such references can now appear.
> >
> > Fixed by reverting that to 4.7 behavior.
> >
>
Ping
C++ has a much more predictable semantics for inline functions, so we no
longer need to define them (especially in header files) with the
'static' specifier. The upshot is that when the compiler fails to
inline a call in a given translation unit, it keeps only one copy in the
entire program, inst
gfortran supports "?" and "=?" as input with namelists (a somewhat
common vendor extension). Either of those can be used with stdin to
print the available fields of the namelist. With non-stdin input, the ?
and =? lines are simply ignored.
However, two patches, one in 2008 and one in 2011 brok
Hi,
in this PR Tom noticed that we are producing weird extra initial spaces
in some error messages and Manuel replied that in fact those messages
should rather be inform. Per Manuel suggestion, I therefore adjusted
call.c in that sense but then noticed that eg, across rv1n.C, we were
producin
committed as revision 197198
kenny
On 03/27/2013 10:19 AM, Richard Biener wrote:
On Wed, Feb 27, 2013 at 1:37 AM, Kenneth Zadeck
wrote:
the original patch 3 was accepted and committed. These were subsequent
comments. This could likely be just checked in the next stage 1 without
approval.
Applied.
Gerald
Index: readings.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/readings.html,v
retrieving revision 1.222
diff -u -3 -p -r1.222 readings.html
--- readings.html 27 Mar 2013 22:10:04 - 1.222
+++ readings.html
>> Note that the testsuite currently fails when executing under qemu:
>> - support of /proc/self/maps does not conform to the kernel format.
>> One extra space is missing from some lines, which confuses libsanitizer.
>> Patch proposed to upstream qemu:
>> http://lists.gnu.org/archive/html/qem
committed as revision 197200.
kenny
On 03/27/2013 11:07 AM, Richard Biener wrote:
On Wed, Mar 27, 2013 at 3:23 PM, Kenneth Zadeck
wrote:
On 03/27/2013 10:18 AM, Richard Biener wrote:
On Wed, Feb 27, 2013 at 1:27 AM, Kenneth Zadeck
wrote:
Here is the second of my wide int patches with the
.. oops, the patch I attached has a typo. This is the right one.
Paolo.
//
Index: cp/call.c
===
--- cp/call.c (revision 197198)
+++ cp/call.c (working copy)
@@ -5836,11 +5836,11 @@ convert_like_real (conversi
On 20/03/13 17:21, Ian Bolton wrote:
MOVK should not be generated with a negative immediate, which
the assembler rightfully rejects.
This patch makes MOVK output its 2nd operand in hex instead.
Tested on bare-metal and linux.
OK for trunk?
Cheers,
Ian
2013-03-20 Ian Bolton
gcc/
richard,
adding the gcc_checking_assert is going to require that i include
system.h in hwint.h which seems to cause a loop. while in principle, i
agree with the assert, this is going to be a mess.
kenny
On 03/27/2013 10:13 AM, Richard Biener wrote:
On Wed, Feb 27, 2013 at 1:22 AM, Kenneth
Ok.
Jason
The domwalker fix to order dom children after inverted postorder
exposed the latent issue that LIM relies on domwalk to walk
all blocks defining SSA names before all blocks using them ...
which is what the following patch tries to fix using the
dependency information it already tracks (but incompl
Hi,
this patch adds streaming of histograms that did not make it into LTO
previously. This restores some optimizations, such as specialization of
memcpy/memset inline code for expected size&alignment of the block.
I also added specialzed streaming functions for gcov counters. They may be
wider th
On Thu, Mar 28, 2013 at 5:34 AM, Wei Mi wrote:
> I am not familiar how to use define_subst, so I write a patch that
> changes define_insn_and_split to define_insn. bootstrapped and
> regression tested on x86_64-unknown-linux-gnu.
>
> A question is: after that change, Is there anyway I can make
> t
I wrote:
Ping**2?
I'd like to get these patches committed, if possible, to clear up my
trees a little bit :-)
Thomas
*ping*
Slightly updated patch below, with a better test case as suggested
by Dominique.
OK for trunk?
2013-03-16 Thomas Koenig
PR fortran/45159
Am 26.03.2013 18:17, schrieb Thomas Koenig:
Hi Tobias,
I have not yet looked at the patch, but I wonder whether that causes
invalid code for
if (any([a,b,c] < f()))
by evaluating f() multiple times.
This is avoided by this part of the patch:
+ if (op2->expr_type == EXPR_CONSTANT)
+sc
Hello Joseph, Aldy et al.,
Attached, please find a fixed patch (bzipped) that implements array
notation for C. To my best knowledge, I have fixed all the changes Joseph and
Aldy have mentioned in the previous email threads
(http://gcc.gnu.org/ml/gcc-patches/2013-03/msg01182.html,
http:/
Hi,
This patch adds support in AArch64 for scalar moves to and from vector
registers in SHORT modes.
This has been tested with a full regression run on aarch64-elf.
OK for trunk and 4.8?
Thanks
Sofiane
-
2013-03-28 Sofiane Naci
* config/aarch64/aarch64.md (*mov_aarch64): Add al
Hi,
This patch adds support in AArch64 for scalar loads and stores to and from
B/H registers.
This has been tested with a full regression run on aarch64-elf.
OK for trunk and 4.8?
Thanks
Sofiane
-
2013-03-28 Sofiane Naci
* config/aarch64/aarch64.md (*mov_aarch64): Add variants
On 26/03/13 11:35, Hurugalawadi, Naveen wrote:
Hi,
>>Can we split them into two different patches. Just and in one
Thanks for reviewing the patch. I have split the patches for "and"
separately as per your suggestion.
Thanks, I've adjusted the patch to use the SHIFT iterator instead of
ASH
On 28/03/13 17:15, Sofiane Naci wrote:
Hi,
This patch adds support in AArch64 for scalar moves to and from vector
registers in SHORT modes.
This has been tested with a full regression run on aarch64-elf.
OK for trunk and 4.8?
OK for trunk.
/Marcus
On 28/03/13 17:15, Sofiane Naci wrote:
Hi,
This patch adds support in AArch64 for scalar loads and stores to and from
B/H registers.
This has been tested with a full regression run on aarch64-elf.
OK for trunk and 4.8?
OK for trunk.
/Marcus
Thomas Koenig wrote:
below is a patch which improves dependency checking for array
assignments and calculation of string lengths.
Talking about dependencies, I wonder whether you would be interested
implementing the function
bool gfc_simply_noncontiguous (gfc_expr *);
or something similar
Currently, if you compile with -fomit-frame-pointer, the frame record
and frame pointer are still maintained (i.e. There is no way to get
the behaviour you are asking for!).
This patch fixes that. It also makes sure that if you ask for no frame
pointers in leaf functions then they are not generat
In this testcase we were crashing because after function argument
substitution in the initialization for crashnkill we couldn't figure out
how to simplify *(Inner*)4, so we went back to bridge2. It certainly
doesn't make sense for the initializer to refer to a parameter for a
function called i
The compiler was getting confused when an implicit capture added a __t
class binding and then tried to pop the __t local variable. But we
don't need to push the closure members as bindings at all, since we now
use capture proxies for name resolution.
The second patch fixes a -Wshadow issue I
When 'this' appears in an expression, it should be an rvalue rather than
a const lvalue; in C++11, the distinction matters.
The cp_build_indirect_ref change is to avoid building extra copies of
*this due to this change.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit a668cad5042d5e499b4
The only valid operand for sizeof... is a single identifier, so it
doesn't really make sense to share code with other forms of sizeof.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit 8a4393fa57437072d140065b5949e3aa9c4674d6
Author: Jason Merrill
Date: Wed Mar 27 17:20:41 2013 -0400
The change to force instantiation to check for abstract return types
broke this change; until the core SG has a chance to discuss the
language inconsistency, I'm going to disable that instantiation in
decltype context.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit d84445fd99912ad25589
Hello!
2013-03-28 Uros Bizjak
* config/i386/i386.md (*vec_extract2vdi_1): Merge with
*vec_extractv2di_1_rex64. Use x64 isa attribute.
Tested on x86_64-pc-linux-gnu {,-m32} and committed to mainline SVN.
Uros.
Index: sse.md
On 28.03.2013 17:11, Thomas Koenig wrote:
Am 26.03.2013 18:17, schrieb Thomas Koenig:
Hi Tobias,
I have not yet looked at the patch, but I wonder whether that causes
invalid code for
if (any([a,b,c] < f()))
by evaluating f() multiple times.
This is avoided by this part of the patch: [...] wh
Hi,
I'm not sure how good the coverage is from these test cases
On 26/03/13 11:42, Hurugalawadi, Naveen wrote:
(define_insn "*add3_compare0"
[(set (reg:CC_NZ CC_REGNUM)
(compare:CC_NZ
-(plus:GPI (match_operand:GPI 1 "register_operand" "%r,r")
+(plus:GPI (match_oper
On 28/03/13 17:55, Ian Bolton wrote:
Currently, if you compile with -fomit-frame-pointer, the frame record
and frame pointer are still maintained (i.e. There is no way to get
the behaviour you are asking for!).
This patch fixes that. It also makes sure that if you ask for no frame
pointers in l
Le 28/03/2013 18:48, Tobias Burnus a écrit :
> Talking about dependencies, I wonder whether you would be interested
> implementing the function
>bool gfc_simply_noncontiguous (gfc_expr *);
> or something similarly named.
>
> It should return true, if the expression is known to be noncontiguous
*ping* / http://gcc.gnu.org/ml/fortran/2013-03/msg00102.html
On March 21, 2013 19:02, Tobias Burnus wrote:
Dear all,
using VALUE, gfortran passes the arguments by value.* That works well,
except if VALUE is combined with OPTIONAL. Currently, "call foo(0)"
and "call foo()" are indistinguisha
Hi Tobias,
I have committed the two approved patches, with the issues that you
have noted fixed.
Thanks a lot for the reviews!
PS: Regarding the string comparison patch: As Janne, think it should be
fine with also supporting kind=4 (don't forget to multiply the length by
the kind, e.g. ts.kind
Hi Tobias,
Talking about dependencies, I wonder whether you would be interested
implementing the function
bool gfc_simply_noncontiguous (gfc_expr *);
or something similarly named.
If you will work on it, please tell me -
Sounds interesting. I'll give it a shot.
Thomas
I'd suggest rewriting this expression in some easier way:
p += (*p == '%' && *(p + 1)) ? 2 : 1;
I'd prefer
if (*p == '%')
p++;
p++;
That's not the same thing though. Maksim's code is correct,
although it could certainly be written more
On 03/28/2013 03:21 AM, Tobias Burnus wrote:
> Tobias Burnus wrote:
>> b) To copy the format string
>>
>> The attached patch does the latter. The current hashing algorithm avoids hash
>> collisions by checking whether the value is exactly the same - and the value
>> is given by the format string. T
On 03/28/2013 06:48 AM, Tobias Burnus wrote:
> gfortran supports "?" and "=?" as input with namelists (a somewhat common
> vendor
> extension). Either of those can be used with stdin to print the available
> fields
> of the namelist. With non-stdin input, the ? and =? lines are simply ignored.
>
The trunk was merged into the c++-concepts branch as of revision 197225.
-- Gaby
On 3/28/13, Richard Biener wrote:
> The patch is ok as-is.
Committed.
--
Lawrence Crowl
Yep, sure. I missed that (*p != NULL) check.
Thanks, Michael
On 29 March 2013 05:15, Segher Boessenkool wrote:
>> I'd suggest rewriting this expression in some easier way:
>> p += (*p == '%' && *(p + 1)) ? 2 : 1;
>>
>> I'd prefer
>> if (*p == '%')
>> p
59 matches
Mail list logo
