> I've backported this fix to 4.8 & 4.9 branch.
> These patches have been tested for armeb-none-eabi-gcc/g++ with qemu, and
> both the test results were ok.
Looks OK with me. Ramana, is this OK for the 4.8 & 4.9 branches? Thanks.
I've backported this fix to 4.8 & 4.9 branch.
These patches have been tested for armeb-none-eabi-gcc/g++ with qemu,
and both the test results were ok.
--- gcc/ChangeLog.ori 2014-11-28 17:30:43.0 +0800
+++ gcc/ChangeLog 2014-11-28 17:30:55.0 +0800
@@ -1,3 +1,15 @@
+2014-11-
Thanks, I've added a testcase and committed this patch. Bootstrapped
with no regressions on x86_64-unknown-linux-gnu.
libcpp:
2014-11-29 John Schmerge
PR preprocessor/41698
* charset.c (one_utf8_to_utf16): Do not produce surrogate pairs
for 0x.
gcc/testsuite:
2014
The attached simple change fixes a long standing regression since
4.2. When we have stack arguments and a sibling
call, the dse pass deletes stores to the parent's stack frame when we
have a sibling call because they are are off frame
for the current function. As a result, the sibling call ar
On Sun, 23 Nov 2014, Tobias Burnus wrote:
> Build on x86-64-gnu-linux with checking that the resulting options.c remains
> the same.
> OK for the trunk?
OK.
--
Joseph S. Myers
jos...@codesourcery.com
Mike Stump wrote:
> While I had testing on my target, there were plenty of other regressions
> from trunk intermixed, so I wanted to do a normal bootstrap on Linux which
> I had not done yet. Kaz, you can check your patch, if the same, pretty
> sure it is, it has been approved and you can just ch
btw, a bit of followup...
There were a few other ports that had checks similar to that in their
handlers.. ie in rs6000.c:rs6000_handle_longcall_attribute() and
i386:ix86_handle_cconv_attribute()
if (TREE_CODE (*node) != FUNCTION_TYPE
&& TREE_CODE (*node) != FIELD_DECL
&& TREE_CO
On Fri, 28 Nov 2014, Kai Tietz wrote:
> 2014-11-28 19:10 GMT+01:00 Joseph Myers :
> > On Fri, 28 Nov 2014, Kai Tietz wrote:
> >
> >> Some diagnostics are different and some constructs getting allowed
> >> with enabled ms-extensions flag. Additionally is the pedantic-flag
> >> not automatically se
The following patch fixes
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64061
The patch was tested and bootstrapped on x86-64.
Committed as rev. 218171.
2014-11-28 Vladimir Makarov
PR target/64061
* lra.c (lra_substitute_pseudo): Ignore constant with int mode for
sub
Hi!
As the following patch shows, , NULL) is passed to these 5 functions
way too often that forcing everybody to spell it everywhere seems to
verbose, leading to formatting issues and making code less readable.
Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?
As possible foll
2014-11-28 19:10 GMT+01:00 Joseph Myers :
> On Fri, 28 Nov 2014, Kai Tietz wrote:
>
>> Some diagnostics are different and some constructs getting allowed
>> with enabled ms-extensions flag. Additionally is the pedantic-flag
>> not automatically set for *-*-mingw* targets. So for enforcing
>> ISO-
On Fri, 28 Nov 2014, Kai Tietz wrote:
> Some diagnostics are different and some constructs getting allowed
> with enabled ms-extensions flag. Additionally is the pedantic-flag
> not automatically set for *-*-mingw* targets. So for enforcing
> ISO-C++ pedantic checks the *-*-mingw* targets need t
2014-11-28 18:53 GMT+01:00 Joseph Myers :
> On Fri, 28 Nov 2014, Kai Tietz wrote:
>
>> Hi,
>>
>> this patch turns off ms-extensions for mingw-targets to match
>> diagnostics checked in testcases.
>>
>> Ok for apply?
>
> For the tests using -std= -pedantic (or
> -pedantic-errors), are you saying the
On Fri, 28 Nov 2014, Kai Tietz wrote:
> Hi,
>
> this patch turns off ms-extensions for mingw-targets to match
> diagnostics checked in testcases.
>
> Ok for apply?
For the tests using -std= -pedantic (or
-pedantic-errors), are you saying the diagnostics are *different*, or that
some construct
On 11/14/2014 10:28 PM, Tobias Burnus wrote:
All in all: Okay when tesing succeeded. I still prefer some words what's
excluded (or included) in minimal as comment in configure.ac, but the
patch is also okay without.
I thought you meant something more than adding a comment. I've added
this in t
On Thu, Nov 27, 2014 at 6:48 AM, Richard Biener wrote:
>
> Currently the order in which patterns in match.pd match is pretty
> random (well, it's deterministic of course). This is bad if there
> are multiple possible matches such as in
>
> z = x + 1;
> w = z + 0;
>
> where we have both z + 0
On 11/06/2014 01:31 PM, Yury Gribov wrote:
On 10/21/2014 07:24 PM, Yury Gribov wrote:
On 10/13/2014 02:26 PM, Yury Gribov wrote:
On 10/02/2014 09:14 PM, Yury Gribov wrote:
On 09/17/2014 09:08 PM, Yury Gribov wrote:
> On 09/16/2014 08:38 PM, Yury Gribov wrote:
>> Hi all,
>>
>> This is the t
Hi!
I've committed following backports to 4.8 branch, after
bootstrapping/regtesting on x86_64-linux and i686-linux.
Jakub
2014-11-28 Jakub Jelinek
Backported from mainline
2014-10-03 Jakub Jelinek
PR libgomp/61200
* omp-low.c (taskreg_contexts): Ne
On 28/11/14 15:24 +, Jonathan Wakely wrote:
Tested x86_64-linux with:
--target_board=unix\{-m32,\}\{,-D_GLIBCXX_USE_CXX11_ABI=0\}
Also tested on powerpc64-linux (where this patch doesn't actually
bootstrap due to PR 63573, there's a patch in the PR that's needed)
and i686-linux.
Oh how s
On 11/26/2014 10:23 AM, Zhenqiang Chen wrote:
> /* Check to make sure CC is not clobbered since prepare_operand might
> generates copy or mode convertion insns, although no test shows
> such insns clobber CC. */
And what do you do if a clobber does happen?
With TER enabled, there's ev
On Nov 28, 2014, at 4:03 AM, Richard Biener wrote:
> On Fri, Nov 28, 2014 at 11:37 AM, Kaz Kojima wrote:
>> Hi,
>>
>> I've got odd ICEs with some bitmap operations when working on
>> sh-lra branch.
>> bitmap_ior and bitmap_ior_and_compl are the bitmap operations
>> for DST = A | B and DST = A |
Hi,
the attached Ada testcase triggers an assertion in the RTL expander for the
address operator because the operator has been applied to a non-byte-aligned
record field. The problematic ADDR_EXPR is built by ipa_modify_call_arguments
which has a hole when get_addr_base_and_unit_offset return
Hi Dodji,
On 23 October 2014 at 13:11, Dodji Seketeli wrote:
> /* If MAP is not the last line map of its set, then the new location
>(loc + offset) should be less than the first location encoded by
>the next line map of the set. */
which is currently implemented as:
if (map
On Tue, Aug 26, 2014 at 03:38:43PM -0400, Jason Merrill wrote:
> commit ccf3f3b41516b34d7d564bed1b3f4e3cf270e43a
> Author: Jason Merrill
> Date: Tue Aug 26 13:56:17 2014 -0400
>
> PR c++/58624
> * pt.c (tsubst_decl) [VAR_DECL]: Copy TLS model.
> (tsubst_copy_and_build) [VAR_DE
On Fri, Nov 28, 2014 at 04:49:28PM +0100, Marek Polacek wrote:
> I forgot to adjust the C++ part of the shift instrumentation as well.
>
> Bootstrapped/regtested on ppc64-linux, ok for trunk?
>
> 2014-11-28 Marek Polacek
>
> * c-ubsan.c (ubsan_instrument_shift): Use op1_utype for MINUS_
I forgot to adjust the C++ part of the shift instrumentation as well.
Bootstrapped/regtested on ppc64-linux, ok for trunk?
2014-11-28 Marek Polacek
* c-ubsan.c (ubsan_instrument_shift): Use op1_utype for MINUS_EXPR
instead of unsigned_type_node.
* c-c++-common/ubsan/s
While going through the attribute tables to sort out separation or trees
and types, I'm seeing a couple of ports with similar handlers that look
a little odd.
the code sequence in the handler looks like:
if (TREE_CODE (*node) != VAR_DECL
&& TREE_CODE (*node) != POINTER_TYPE
&& TREE
The following patch fixes
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64087
The patch was bootstrapped and tested on x86-64.
Committed as rev. 218162.
2014-11-28 Vladimir Makarov
PR rtl-optimization/64087
* lra-lives.c (process_bb_lives): Add debug output.
(lra_cre
Hi All,
Here is a simple fix for PR 63941 - wrong assertion for dominating bb
which may have true predicate. I deleted it and add setting of
non-true predicate for join bb. After this fix test-case is compiled
successfully.
Bootstrap and regression testing did not show any new failures.
Is it OK
Ping.
On Fri, Nov 21, 2014 at 06:55:51PM +0100, Marek Polacek wrote:
> I noticed that C++14 [basic.types] says that "a type is a literal type
> if it is: void, [...]". Yet our literal_type_p doesn't consider void
> type as a literal type. The following is an attempt to fix that along
> with a te
Here's the next version of the std::string rework, this time with
57.1428% more locale facets!
The problem is that there are several facet types that use std::string
in their public interfaces and as the parameters (or worse, as return
types) of virtual functions. It's not possible to overload the
Hello!
> As discussed for AVX512F submission, using -D in dg-options,
> should be replaced with parts of the testcase itself. Patch below
> does that.
> Ok for trunk?
>
> P. S.
> As patch itself is hard to review, this was done by sed like this:
>
> sed -i 's/ -DAVX512IFMA//g' ../gcc/testsui
On Fri, Nov 28, 2014 at 2:54 PM, Petr Murzin wrote:
> Hello,
> This patch is according to this input:
> https://gcc.gnu.org/ml/gcc-patches/2014-10/msg02441.html
> Furthermore, the following regexps were fixed in order to make more
> precise assembler scanning. The endings like (?:\n|\[\ \t\]+#) ar
Currently check_GNU_style.sh gives the error "There should be exactly one space
between function name and parentheses." for the following kind of lines:
tab[(int) idx]
This patch changes the check to only warn if there is 0 of 2+ space(s) between
a alphanumeric character and an opening parenthe
On Sun, Nov 23, 2014 at 7:47 PM, Zhenqiang Chen wrote:
>
>> -Original Message-
>> From: Richard Henderson [mailto:r...@redhat.com]
>> Sent: Friday, November 21, 2014 2:27 AM
>> To: Zhenqiang Chen; gcc-patches@gcc.gnu.org
>> Subject: Re: [PATCH, ifcvt] Fix PR63917
>>
>> On 11/20/2014 10:48
On Wed, Nov 26, 2014 at 11:18:11AM -0500, Jason Merrill wrote:
> On 10/28/2014 08:44 AM, Jakub Jelinek wrote:
> >+cp_ubsan_check_member_access_r (tree *stmt_p, int *walk_subtrees, void
> >*data)
>
> This function needs a longer comment about exactly what forms it's trying to
> instrument.
Ok, wi
On 11/27/2014 09:26 AM, Marek Polacek wrote:
On Wed, Nov 26, 2014 at 11:56:26AM -0500, Jason Merrill wrote:
Please give diagnostics explaining what's wrong with the shift rather than
the generic "is not a constant expression".
Done.
I was thinking even more detailed: one diagnostic for negat
> Jeff Law wrote:
> Do you have a testcase that shows the expected improvements from this
> change? It's OK if it's specific to a target.
>
> Have you bootstrapped and regression tested this change?
>
> With a test for the testsuite and assuming it passes bootstrap and
> regression testing, this
Hello,
This patch is according to this input:
https://gcc.gnu.org/ml/gcc-patches/2014-10/msg02441.html
Furthermore, the following regexps were fixed in order to make more
precise assembler scanning. The endings like (?:\n|\[\ \t\]+#) are
used in ICC which may generate comments. Please have a look.
On Fri, Nov 28, 2014 at 3:23 AM, Jakub Jelinek wrote:
> On Fri, Nov 28, 2014 at 11:19:18AM +, renlin at gcc dot gnu.org wrote:
>> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63661
>>
>> --- Comment #25 from renlin at gcc dot gnu.org ---
>> Author: renlin
>> Date: Fri Nov 28 11:18:47 2014
>> N
Hi!
I've backported a few patches of mine to 4.9 branch,
bootstrapped/regtested on x86_64-linux and i686-linux,
committed to 4.9 branch.
Jakub
2014-11-28 Jakub Jelinek
Backported from mainline
2014-10-31 Jakub Jelinek
PR rtl-optimization/63659
* ree
On Wed, Nov 12, 2014 at 2:54 PM, Yuri Rumyantsev wrote:
> Hi All,
>
> Here is the last part of patch which is related to vectorization
> improvements, namely
>
> 1. New sub-phase is added which delete dead predicate computations
> (such code can prevent loop vectorization since dead computation a
Kai Tietz writes:
> 2014-11-28 12:14 GMT+01:00 Rainer Orth :
>> Hi Kai,
>>
>>> 2014-11-28 Kai Tietz
>>>
>>> * gcc.dg/anon-struct-1.c:
>>> * gcc.dg/anon-struct-11.c:
>>> * gcc.dg/anon-struct-2.c:
>>> * gcc.dg/c11-anon-struct-2.c:
>>> * gcc.dg/c11-anon-struct-3.c:
>>
>> those
Hi Kai,
> 2014-11-28 12:21 GMT+01:00 Rainer Orth :
>> Hi Kai,
>>
>>> this patch skips some test, which are trying to test non-existing
>>> weak-variant for mingw-targets.
>>
>> why is this necessary when most (all?) of those tests already have
>> dg-require-weak? If there's some property of weak
2014-11-28 12:17 GMT+01:00 Rainer Orth :
> Hi Kai,
>
>> 2014-11-28 Kai Tietz
>>
>> * g++.dg/abi/mangle13.C: Disable
>> ms-extensions for mingw-targets.
>
> just a nit: why break this line way before 72/80 charackters? Several
> more of those useless linebreaks below. And no dash in min
2014-11-28 12:14 GMT+01:00 Rainer Orth :
> Hi Kai,
>
>> 2014-11-28 Kai Tietz
>>
>> * gcc.dg/anon-struct-1.c:
>> * gcc.dg/anon-struct-11.c:
>> * gcc.dg/anon-struct-2.c:
>> * gcc.dg/c11-anon-struct-2.c:
>> * gcc.dg/c11-anon-struct-3.c:
>
> those are not exactly useful ChangeLog
On Wed, Nov 12, 2014 at 2:35 PM, Yuri Rumyantsev wrote:
> Hi All,
>
> Here is the second patch related to extended predication.
> Few comments which explain a main goal of design.
>
> 1. I don't want to insert any critical edge splitting since it may
> lead to less efficient binaries.
> 2. One spe
2014-11-28 12:21 GMT+01:00 Rainer Orth :
> Hi Kai,
>
>> this patch skips some test, which are trying to test non-existing
>> weak-variant for mingw-targets.
>
> why is this necessary when most (all?) of those tests already have
> dg-require-weak? If there's some property of weak variables that
> P
Richard Biener wrote:
> Huh? Wasn't this fixed by Mike a few weeks ago?
Ugh. I've missed that thread. Sorry for the noise.
Regards,
kaz
On Fri, Nov 28, 2014 at 12:33 PM, Evgeny Stupachenko wrote:
> Hi,
>
> After fixing PR60451 (pack scheme instead of pshufb) general power of
> 2 permutations become more profitable.
> So I'd like to revert the changes keeping algorithm live or implement
> new hook to use best algorithm/size for par
On Fri, Nov 28, 2014 at 11:37 AM, Kaz Kojima wrote:
> Hi,
>
> I've got odd ICEs with some bitmap operations when working on
> sh-lra branch.
> bitmap_ior and bitmap_ior_and_compl are the bitmap operations
> for DST = A | B and DST = A | (B & ~KILL) respectively. It
> looks they could make bitmaps
Hi,
After fixing PR60451 (pack scheme instead of pshufb) general power of
2 permutations become more profitable.
So I'd like to revert the changes keeping algorithm live or implement
new hook to use best algorithm/size for particular -mtune.
Below is reverting patch. Bootstrap and make check pass
On 28/11/14 09:20, Richard Biener wrote:
On Thu, Nov 27, 2014 at 5:32 PM, Renlin Li wrote:
Hi all,
PR63661 is effectively fixed by my previous patch here:
https://gcc.gnu.org/ml/gcc-patches/2014-11/msg02253.html
This patch add a test case for it.
Okay for trunk?
Ok.
Thanks,
RIchard.
gcc/
On Fri, Nov 28, 2014 at 11:19:18AM +, renlin at gcc dot gnu.org wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63661
>
> --- Comment #25 from renlin at gcc dot gnu.org ---
> Author: renlin
> Date: Fri Nov 28 11:18:47 2014
> New Revision: 218144
>
> URL: https://gcc.gnu.org/viewcvs?rev=
Hi Kai,
> this patch skips some test, which are trying to test non-existing
> weak-variant for mingw-targets.
why is this necessary when most (all?) of those tests already have
dg-require-weak? If there's some property of weak variables that
PE-COFF lacks, I'd rather have a new effective-target
Hi Kai,
> 2014-11-28 Kai Tietz
>
> * g++.dg/abi/mangle13.C: Disable
> ms-extensions for mingw-targets.
just a nit: why break this line way before 72/80 charackters? Several
more of those useless linebreaks below. And no dash in mingw targets;
better yet use *-*-mingw* instead of para
Hi Kai,
> 2014-11-28 Kai Tietz
>
> * gcc.dg/anon-struct-1.c:
> * gcc.dg/anon-struct-11.c:
> * gcc.dg/anon-struct-2.c:
> * gcc.dg/c11-anon-struct-2.c:
> * gcc.dg/c11-anon-struct-3.c:
those are not exactly useful ChangeLog entries ;-)
Rainer
--
Hi,
this patch skips some test, which are trying to test non-existing
weak-variant for mingw-targets.
ChangeLog
2014-11-28 Kai Tietz
* g++.dg/abi/anon2.C: Skip for mingw targets.
* g++.dg/abi/anon3.C: Likewise.
* g++.dg/abi/thunk5.C: Likewise.
* g++.dg/abi/rtti1.C: Likewise.
Hi,
this patch fixes for mingw-targets some LP assumptions, adjusts for
some testcases that ms-extensions is turned off for diagnostics, and
turns on pedantic for some testcase to match expected diagnostics.
Ok for apply?
Kai
ChangeLog
2014-11-28 Kai Tietz
* g++.dg/abi/mangle13.C: Disa
On 28/11/14 10:49 +, Jonathan Wakely wrote:
I think it's fine to say you can't have more than 2^31 open message
catalogs, but we should for numeric_limits::max() before
... but we should *check* for ...
doing anything and return -1 to say it couldn't be opened.
Hi,
this patch turns off ms-extensions for mingw-targets to match
diagnostics checked in testcases.
Ok for apply?
Kai
ChangeLog
2014-11-28 Kai Tietz
* gcc.dg/anon-struct-1.c:
* gcc.dg/anon-struct-11.c:
* gcc.dg/anon-struct-2.c:
* gcc.dg/c11-anon-struct-2.c:
* gcc.dg/c11
On 27/11/14 23:19 +0100, François Dumont wrote:
Hi
Here is a revisited version. I finally go without compiling in
C++11 as I prefer to use __builtin_alloca instead of using
std::unique_ptr and heap memory. I also realized that I could use
codecvt::max_length to know the max number of multi
Hi,
I've got odd ICEs with some bitmap operations when working on
sh-lra branch.
bitmap_ior and bitmap_ior_and_compl are the bitmap operations
for DST = A | B and DST = A | (B & ~KILL) respectively. It
looks they could make bitmaps with the wrong "current" pointer.
One small example is
DST: [(bi
Tobias Burnus wrote:
> Actually, I wonder whether that can lead to printing an error message
> multiple times.
This already occurs, see pr44978.
Dominique
On 27/11/14 23:53, Gerald Pfeifer wrote:
On Thursday 2014-11-27 11:09, Kyrill Tkachov wrote:
Thanks for the review. The invoke.texi changes were posted and
approved at https://gcc.gnu.org/ml/gcc-patches/2014-11/msg02683.html
and I'm attaching the proposed changes.html patch.
Looks good.
Note
On Fri, Nov 28, 2014 at 12:45 AM, Ilya Verbin wrote:
> On 27 Nov 16:00, Richard Biener wrote:
>> On Thu, Nov 27, 2014 at 2:31 PM, Ilya Verbin wrote:
>> > On 25 Nov 10:51, Richard Biener wrote:
>> >> In the patch adding flag_generate_offload sounds like a good solution,
>> >> I didn't like emittin
Hi,
this patch fixes a few glitches in the post-reload compare elimination pass,
most notably the slightly disturbing opening comment:
This pass assumes:
[...]
(1) All comparison patterns are represented as
[(set (reg:CC) (compare:CC (reg) (immediate)))]
the mode mismatch in be
On Thu, Nov 27, 2014 at 6:08 PM, Martin Liška wrote:
> On 11/21/2014 04:21 PM, Martin Liška wrote:
>>
>> On 11/21/2014 04:02 PM, Richard Biener wrote:
>>>
>>> On Fri, Nov 21, 2014 at 3:39 PM, Martin Liška wrote:
>>>
Hello.
Ok, this is simplified, one can use sreal a = 12345 and it
Hi,
This patch converts vpmaxX & vpminX intrinsics to use builtin functions
instead of the previous inline assembly syntax.
Regtested with aarch64-linux-gnu on QEMU. Also passed the glorious testsuite
of Christophe Lyon.
OK for the trunk?
Index: gcc/ChangeLog
On Thu, Nov 27, 2014 at 5:32 PM, Renlin Li wrote:
> Hi all,
>
> PR63661 is effectively fixed by my previous patch here:
> https://gcc.gnu.org/ml/gcc-patches/2014-11/msg02253.html
>
> This patch add a test case for it.
>
> Okay for trunk?
Ok.
Thanks,
RIchard.
> gcc/testsuite/ChangeLog:
>
> 2014-
On Fri, Nov 28, 2014 at 02:52:11AM +0300, Ilya Verbin wrote:
> On 21 Nov 21:36, Jakub Jelinek wrote:
> > On Fri, Nov 21, 2014 at 11:19:26PM +0300, Ilya Verbin wrote:
> > > '#pragma omp critical (name)' can be placed in the function, marked
> > > with '#pragma omp declare target', in this case the c
On Fri, 28 Nov 2014, Ilya Verbin wrote:
> On 11 Oct 18:49, Ilya Verbin wrote:
> > (run_gcc): Outline options handling into the new functions:
> > find_and_merge_options, append_compiler_options, append_linker_options.
> > ...
> > @@ -625,18 +893,13 @@ run_gcc (unsigned argc, char *argv[])
Hi,
An internal user reported that -march=armv8-a+crc wasn't allowing the
driver to pick up the right multilib for an arm-none-eabi toolchain
configured with --with-multilib-list=aprofile.
Without this patch on arm-none-eabi with
--with-multilib-list=aprofile prints.
$)./xgcc -S -O2 -
Hi Ramana,
On 11/27/2014 11:36 AM, Ramana Radhakrishnan wrote:
On Wed, Nov 19, 2014 at 2:54 PM, Christian Bruel wrote:
On 11/19/2014 03:18 PM, Ramana Radhakrishnan wrote:
On Wed, Nov 19, 2014 at 1:24 PM, Christian Bruel
wrote:
I think I missed the stage3, Anyway would it be OK for stage
74 matches
Mail list logo
