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
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 -
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[])
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 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-
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 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 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 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
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
Tobias Burnus wrote:
> Actually, I wonder whether that can lead to printing an error message
> multiple times.
This already occurs, see pr44978.
Dominique
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
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,
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 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 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
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 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 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,
> 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
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=
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/
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 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
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
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
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
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: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
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
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
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
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 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
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.
> 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
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
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 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
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 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
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
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
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
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
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
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
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
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_
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
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
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
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 |
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 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
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 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
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/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 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
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:
> 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 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-
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
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
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
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
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
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
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
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
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-
> 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.
74 matches
Mail list logo
