On Fri, Dec 11, 2015 at 10:10 AM, Martin Liška wrote:
> Hello.
>
> I've just applied suggested change that Richi proposed.
> The patch can bootstrap on x86_64-linux-gnu and survives regression tests.
> Moreover,
> the memory leak/invalid read has gone.
>
> Ready for trunk?
Ok.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68064
--- Comment #7 from Markus Trippelsdorf ---
The issue seems to be fixed. Not sure what revision fixed it.
LTO bootstrapped on x86_64-unknown-linux-gnu, applied.
Richard.
2015-12-11 Richard Biener
* lto-streamer.h (lto_simple_header_with_strings): Remove
main_size field already in lto_simple_header.
Index: gcc/lto-streamer.h
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68842
Jonathan Wakely changed:
What|Removed |Added
Keywords||diagnostic
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68636
Martin Jambor changed:
What|Removed |Added
CC||trippels at gcc dot gnu.org
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68851
--- Comment #2 from Martin Liška ---
Caused by r231540:
commit 15a1fce36358508909f2013fd6d07e0b9fcad97a
Author: hubicka
Date: Thu Dec 10 19:20:14 2015 +
* ipa-cp.c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68851
Bug ID: 68851
Summary: [6 Regression] ICE: in set_comdat_group, at
ipa-comdats.c:213
Product: gcc
Version: 6.0
Status: UNCONFIRMED
Severity: normal
On Thu, Dec 10, 2015 at 6:05 PM, Sebastian Pop wrote:
> also update ISL to isl as requested by its author Sven Verdoolaege.
Ok. Please always post ChangeLog entries as well.
Richard.
> ---
> gcc/doc/install.texi | 9 +
> gcc/doc/invoke.texi | 4 ++--
> 2 files
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68649
--- Comment #11 from Joost VandeVondele
---
(In reply to Jerry DeLisle from comment #10)
> This PR is tagged as a regression. Has anyone determined when it last
> worked or is it longstanding bug uncovered by recent non-fortran fe changes?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64132
li xin changed:
What|Removed |Added
CC||lixin.fnst at cn dot
fujitsu.com
--- Comment
On Fri, 11 Dec 2015, Jan Hubicka wrote:
> >
> > We explicitely do not use debug-info-level tests in free-lang-data
> > to allow mixing -g and -g0 objects. Are you sure doing the above
> > doesn't mess up tree merging enough to effectively enlarge WPA
> > memory use and the merged decl sections?
On Thu, Dec 10, 2015 at 9:08 PM, Jeff Law wrote:
> On 12/03/2015 07:38 AM, Richard Biener wrote:
>>
>> This pass is now enabled by default with -Os but has no limits on the
>> amount of
>> stmts it copies.
>
> The more statements it copies, the more likely it is that the path
On 12/04/2015 06:15 PM, Dominik Vogt wrote:
> Version 5 with the latest requested changes. Seems to work now.
> I've dropped the extra patch and rather marked the failing tests
> as "xfail".
>
> Ciao
>
> Dominik ^_^ ^_^
>
Patch applied with minor changes:
> + ; Convert Pmode to BLKmode
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64818
Tejas Belagod changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
CC|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68064
--- Comment #8 from Martin Jambor ---
Author: jamborm
Date: Fri Dec 11 11:27:11 2015
New Revision: 231559
URL: https://gcc.gnu.org/viewcvs?rev=231559=gcc=rev
Log:
Add an asssert and testcase for PR 68064
2015-12-11 Martin Jambor
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68139
--- Comment #3 from ville at gcc dot gnu.org ---
Author: ville
Date: Fri Dec 11 12:04:23 2015
New Revision: 231562
URL: https://gcc.gnu.org/viewcvs?rev=231562=gcc=rev
Log:
PR libstdc++/68139
Added:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64132
--- Comment #7 from li xin ---
(In reply to li xin from comment #6)
> The LSB test cases also Failed.
> /22_locale/messages/members/char/2.cc 1
> /22_locale/messages/members/char/wrapped_env.cc 1
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68835
Jakub Jelinek changed:
What|Removed |Added
CC||rsandifo at gcc dot gnu.org
Hi,
This patch adds a parameter to TERMINATE_PASS_LIST, that should match
the pass list it's supposed to terminate.
The intention of the patch is that it:
- makes it easier to understand the top-level hierarchy of the pass
list (given that the top-level list may be quite long).
- ensures
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68064
Martin Jambor changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68853
Markus Trippelsdorf changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
Ping.
https://gcc.gnu.org/ml/gcc-patches/2015-12/msg00526.html
Thanks,
Kyrill
On 04/12/15 09:30, Kyrill Tkachov wrote:
Hi all,
We don't handle properly the patterns for the [us]bfiz and [us]bfx instructions
when they
have an extend+ashift form. For example, the
*_ashl pattern.
This leads to
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68852
Bug ID: 68852
Summary: [6 Regression] ICE in vect_is_simple_use
(tree-vect-stmts.c:8750) using -O3
Product: gcc
Version: 6.0
Status: UNCONFIRMED
Severity:
On Wed, Dec 9, 2015 at 4:27 PM, Yvan Roux wrote:
> Hi,
>
> as it was raised in
> https://gcc.gnu.org/ml/gcc-patches/2015-10/msg01540.html we experiment
> random failures in gfortran validation when it is run in parallel (-j
> 8). The issues occurs because of concurrent
Hi,
PR 68064 has been fixed by Richi's revision 231246. I would still
like to add the testcase to the testsuite and add a checking assert
so that if ever get zero alignment again, we catch it in the analysis
part of IPA-CP (which with LTO means in compilation and not linking
phase which makes a
On Thu, Dec 10, 2015 at 05:56:26PM +0100, Jan Hubicka wrote:
> > Is this really necessary, is it not enough to remove the assignment to
> > ret below? If the parameter is not used, devirtualization time bonus,
> > which you then rely on estimate_local_effects, should be zero for it.
> >
> > It
On Fri, Dec 11, 2015 at 10:58 AM, Ilya Enkovich wrote:
> Hi,
>
> Currently when MASK_LOAD and MASK_STORE is vectorized we check
> scalar type of a mask but don't check its vector mask. It means
> we may vectorize it when mask was just loaded from an array of
> booleans.
On Fri, Dec 11, 2015 at 11:00 AM, Tom de Vries wrote:
> Hi,
>
> this patch removes a TERMINATE_PASS_LIST from passes.def that is not paired
> with any INSERT_PASSES_AFTER.
>
> Bootstrapped and reg-tested on x86_64.
>
> OK for stage3 trunk?
Ok.
Richard.
> Thanks,
> - Tom
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68744
--- Comment #3 from Dominique d'Humieres ---
Is this PR fixed by revision r231485?
On 04/12/15 12:39, Szabolcs Nagy wrote:
As described in pr other/67627, the all-multi target can be
built in parallel with the %_.lo targets which generate make
dependencies that are parsed during the build of all-multi.
gcc -MD does not generate the makefile dependencies in an
atomic way so
On 11/12/15 09:53 +0200, Ville Voutilainen wrote:
On 11 December 2015 at 09:52, Marc Glisse wrote:
/libstdc++-v3
* libsupc++/nested_exception.h (_S_rethrow): Use __std::addressof.
Typo.
I must be blind, but I don't see what you mean. :)
Shouldn't the underscores
> On 11 Dec 2015, at 11:25, Dominique d'Humières wrote:
>
> This breaks bootstrap on darwin:
>
> ../../work/gcc/config/darwin.c: In function 'bool
> darwin_use_anchors_for_symbol_p(const_rtx)':
> ../../work/gcc/config/darwin.c:3016:9: error: statement is indented as if it
> > For now I added the information if section is compressed into
> > decl_state. I am not thrilled by this but it is only way I found w/o
> > wasting 4 bytes per every lto section (because the lto header is not
> > really extensible and the stream is assumed to be aligned).
>
> So this trick
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68849
Eric Botcazou changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
On Thu, Dec 10, 2015 at 8:33 PM, David Edelsohn wrote:
> On Thu, Dec 10, 2015 at 2:23 PM, Bill Schmidt
> wrote:
>> Hi,
>>
>> The subject test case has been failing as follows:
>>
>> FAIL: gcc.dg/vect/costmodel/ppc/costmodel-fast-math-vect-pr29925.c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66551
Eric Botcazou changed:
What|Removed |Added
Status|WAITING |RESOLVED
Resolution|---
On Fri, 11 Dec 2015, Jan Hubicka wrote:
> Hi,
> this patch makes WPA to copy sections w/o decompressing them. This leads
> to a nice /tmp usage for GCC bootstrap (about 70%) and little for Firefox.
> In GCC about 5% of the ltrans object file is the global decl section, while
> for Firefox it is
On Thu, Dec 10, 2015 at 6:05 PM, Sebastian Pop wrote:
> we now check the isl version, as there are no real differences in existing
> files
> in between isl 0.14 and isl 0.15.
I thought ISL 0.15 has some new features you could check? Also using a run test
is bad for cross
Hi,
Currently when MASK_LOAD and MASK_STORE is vectorized we check
scalar type of a mask but don't check its vector mask. It means
we may vectorize it when mask was just loaded from an array of
booleans. This happens e.g. for following test:
SUBROUTINE TEST (x, y, z, mask, ims, ime)
IMPLICIT
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68842
--- Comment #2 from Jonathan Wakely ---
And interestingly EDG compiles it without error!
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68851
--- Comment #1 from Markus Trippelsdorf ---
Still reducing. But Martin emailed me a patch, that fixes the issue:
diff --git a/gcc/ipa-comdats.c b/gcc/ipa-comdats.c
index 3aabd71..f7a6ff4 100644
--- a/gcc/ipa-comdats.c
+++ b/gcc/ipa-comdats.c
@@
Hi,
this patch removes a TERMINATE_PASS_LIST from passes.def that is not
paired with any INSERT_PASSES_AFTER.
Bootstrapped and reg-tested on x86_64.
OK for stage3 trunk?
Thanks,
- Tom
Remove unpaired TERMINATE_PASS_LIST in passes.def
2015-12-11 Tom de Vries
*
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68852
Richard Biener changed:
What|Removed |Added
Status|UNCONFIRMED |ASSIGNED
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68850
Dominique d'Humieres changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
On 12/04/2015 06:12 PM, Dominik Vogt wrote:
> Version 6 with another fix. This should work now.
Applied. Thanks!
-Andreas-
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68853
--- Comment #2 from Markus Trippelsdorf ---
g++ -MMD -MF
obj/third_party/WebKit/Source/core/layout/webcore_rendering.LayoutObject.o.d
-DV8_DEPRECATION_WARNINGS -DCLD_VERSION=2 -D_FILE_OFFSET_BITS=64 -DDISABLE_NACL
-DCHROMIUM_BUILD
On Fri, Dec 11, 2015 at 12:03:26AM +, Joseph Myers wrote:
> On Thu, 10 Dec 2015, Manuel López-Ibáñez wrote:
> > On 12/09/2015 03:53 PM, Tobias Burnus wrote:
> > > In principle, %<%c%> and %<%d%> should be convertable to %qc and
> > > %qd (as the code is more readable), but the current function
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68851
Richard Biener changed:
What|Removed |Added
Target Milestone|--- |6.0
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68846
Dominique d'Humieres changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68853
Bug ID: 68853
Summary: [6 Regression] gcc-6 miscompiles Chromium v8 garbage
collector
Product: gcc
Version: 6.0
Status: UNCONFIRMED
Severity: normal
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67484
--- Comment #10 from Martin Liška ---
Author: marxin
Date: Fri Dec 11 10:59:29 2015
New Revision: 231556
URL: https://gcc.gnu.org/viewcvs?rev=231556=gcc=rev
Log:
Fix PR target/67484
Uros Bizjak
PR
This breaks bootstrap on darwin:
../../work/gcc/config/darwin.c: In function 'bool
darwin_use_anchors_for_symbol_p(const_rtx)':
../../work/gcc/config/darwin.c:3016:9: error: statement is indented as if it
were guarded by... [-Werror=misleading-indentation]
return
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67992
Jan Stancek changed:
What|Removed |Added
CC||jan.stancek at gmail dot com
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68139
Ville Voutilainen changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
Hi,
while testing the oacc kernels patch series on top of trunk, using the
optimal handling of BUILTIN_IN_GOACC_PARALLEL in fipa-pta I ran into a
failure where the stores to the omp_data_sizes array were removed by dse.
The call bb in the failing testcase normally looks like this:
...
:
Hi,
This patch adds Pointer Bounds Checker support for VA_ARG calls. I added
bndret call for VA_ARG and corresponding bndret replacement when VA_ARG is
expanded. This fixes all vararg tests from MPX testsuite. Bootstrapped and
tested on x86_64-pc-linux-gnu. Will commit after additional
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68820
Eric Botcazou changed:
What|Removed |Added
CC||ebotcazou at gcc dot gnu.org
---
On Thu, Dec 10, 2015 at 11:03 PM, Jason Merrill wrote:
> The C++ front end uses a temporary hash table to remember specializations of
> local variables during template instantiations. In a nested function such
> as a lambda or local class member function, we need to retain the
H Joern,
> Or have some target hook to make it not even bother filling delay slots
> speculatively; for targets that can fully unexpose the delay slot, like SH and
> ARC >= ARC700, this aspect of fill_eager_delay_slots only mucks up
> schedules and increases code size.
I propose to solve the
On Fri, 11 Dec 2015, Shiva Chen wrote:
> Hi all,
>
> Loop unrolling would decrease performance in some case due to unrolling high
> register pressure
> loops and produce lots of spill code.
>
> I try to implement register pressure aware loop
> unrolling(-funroll-loops-pressure-aware)
> to
On 10/12/15 17:58 -0500, DJ Delorie wrote:
I can't really judge this one. Either DJ or Jon would need to some in
on this.
Looks OK to me, although in the default configuration (plain DJGPP)
the #ifdefs will always be false (omitted), which is harmless.
Is there a non-default configuration
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68851
--- Comment #3 from Martin Liška ---
So the mentioned revision is responsible for creation of a new consprop clone:
IPA decision stage:
...
- Creating a specialized node of virtual bool
blink::HTMLFormControlElement::isFormControlElement()
Function (or more narrow) scope static variables (as well as others not
placed on the stack) should also not have any effect on the stack
alignment. I noticed the issue first with Linux'es dynamic_pr_debug()
construct using an 8-byte aligned sub-file-scope local variable.
According to my checking
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68856
Bug ID: 68856
Summary: wrong compilation wtih character interoperability
Product: gcc
Version: 5.3.1
Status: UNCONFIRMED
Severity: normal
Priority: P3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68854
Bug ID: 68854
Summary: isystem and ansi cause arm assembly preprocessing
problem
Product: gcc
Version: 5.2.1
Status: UNCONFIRMED
Severity: normal
The attached patch introduces wide int support to S/390 in order
to resolve a test case failure in gcc.dg/pr68129_1.c that is
caused by an assertion in
simplify-rtx.c:simplify_const_binary_operation().
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
gcc/ChangeLog
*
On 13/11/15 12:39, Jakub Jelinek wrote:
We simply have some compiler internal interface between the caller and
callee of the outlined regions, each interface in between those has
its own structure type used to communicate the info;
we can attach attributes on the fields, or some flags to
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68842
--- Comment #3 from Jonathan Wakely ---
--- a/gcc/cp/typeck.c
+++ b/gcc/cp/typeck.c
@@ -4907,8 +4907,13 @@ cp_build_binary_op (location_t location,
if (!result_type)
{
if (complain & tf_error)
- error ("invalid operands of
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68853
--- Comment #4 from Markus Trippelsdorf ---
(In reply to Richard Biener from comment #3)
> Can you bisect it to the alias changes from Honza or is this older?
The first issue is older. At least a few weeks.
I don't have a powerful enough
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68381
ktkachov at gcc dot gnu.org changed:
What|Removed |Added
Status|NEW |RESOLVED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68853
--- Comment #5 from Markus Trippelsdorf ---
(In reply to Markus Trippelsdorf from comment #4)
> (In reply to Richard Biener from comment #3)
> > Can you bisect it to the alias changes from Honza or is this older?
>
> The first issue is older.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67484
Martin Liška changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
On 12/11/2015 02:22 AM, Eric Botcazou wrote:
This patch allows a target to increase the cost of anti-deps to better
reflect the actual cost on the machine.
But it can already do it via the TARGET_SCHED_ADJUST_COST hook, can't it?
And can't this be done with define_bypass as well?
Jeff
I noticed a C++ test ICE the compiler trying to output a 128bit enum
initializer. The fix is crazy simple -- have nvptx_assemble_initializer return
false rather than ICE. Defining SUPPORTS_WIDE_INT is helpful, but not necessary.
While adding the testcase, I noticed I'd missed the opening
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68543
--- Comment #3 from ktkachov at gcc dot gnu.org ---
After some discussion on IRC, WORD_REGISTER_OPERATIONS seems wrong for aarch64
since 32-bit operations i.e. in SImode operate like normal 32-bit operations
because they use the 32-bit W-form of
Dominik Vogt wrote:
> +; Note: Although CONST_INT and CONST_DOUBLE are not handled in this
> predicate,
> +; at least one of them needs to appear or otherwise safe_predicate_mode will
> +; assume that a DImode LABEL_REF is not accepted either (see genrecog.c).
The problem is not DImode
On 10/12/15 14:02 +, Jonathan Wakely wrote:
This adjusts some Doxygen comments and updates the Doxygen config file
to ensure all headers are processed (previously doxygen was ignoring
filenames without an extension, which is a lot of the library!)
There's a workaround in include/std/bitset
On Fri, 11 Dec 2015, Tom de Vries wrote:
> On 13/11/15 12:39, Jakub Jelinek wrote:
> > We simply have some compiler internal interface between the caller and
> > callee of the outlined regions, each interface in between those has
> > its own structure type used to communicate the info;
> > we can
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68775
--- Comment #8 from Richard Biener ---
Ok, it looks like a ppc64le cross happily (eh) accepts sources preprocessed on
x86_64-linux and even required built modules.
So I have the dump files myself and the -fopt-info-vec difference (for BB
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68855
Bug ID: 68855
Summary: PAREN_EXPR not "ignored" where possible
Product: gcc
Version: 6.0
Status: UNCONFIRMED
Keywords: missed-optimization
Severity: normal
On 12/11/2015 03:20 PM, Dominik Vogt wrote:
> The attached patch introduces wide int support to S/390 in order
> to resolve a test case failure in gcc.dg/pr68129_1.c that is
> caused by an assertion in
> simplify-rtx.c:simplify_const_binary_operation().
Applied with the change suggested by Uli.
On Fri, 11 Dec 2015, Tom de Vries wrote:
> Hi,
>
> while testing the oacc kernels patch series on top of trunk, using the optimal
> handling of BUILTIN_IN_GOACC_PARALLEL in fipa-pta I ran into a failure where
> the stores to the omp_data_sizes array were removed by dse.
>
> The call bb in the
On 11/12/15 12:47 +, Jonathan Wakely wrote:
* doc/doxygen/user.cfg.in: Use EXTENSION_MAPPING tag. Add new headers
to INPUT. Remove obsolete XML_SCHEMA and XML_DTD tags. Update
PREDEFINED macros. Set BRIEF_MEMBER_DESC for man-pages.
Oops, that changelog is not quite
Richard.
Thanks for your review.
I re-designed fix for assert by adding additional checks for vector
comparison with boolean result to fold_binary_op_with_conditional_arg
and remove early exit to combine_cond_expr_cond.
Unfortunately, I am not able to provide you with test-case since it is
in my
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68853
Richard Biener changed:
What|Removed |Added
Target Milestone|--- |6.0
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68775
--- Comment #7 from Richard Biener ---
Created attachment 36993
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=36993=edit
candidate patch
Can you try if the fix for PR68852 (attached) fixes this?
If not, can you attach
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68835
--- Comment #9 from Jakub Jelinek ---
Testcase that is miscompiled starting with r210113 at -O2:
__attribute__((noinline, noclone)) unsigned __int128
foo (void)
{
unsigned __int128 x = (unsigned __int128) 0xULL;
struct {
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68835
--- Comment #10 from Jakub Jelinek ---
Seems get_int_cst_ext_nunits already has some code to handle the case of
"negative" extra large integers, but only if they have precision multiple of
HOST_BITS_PER_WIDE_INT. So I believe we want something
On 08 Dec 13:53, Aleksandra Tsvetkova wrote:
> Wrong version of patch was attached.
>
> On Tue, Dec 8, 2015 at 1:46 PM, Aleksandra Tsvetkova
> wrote:
> > gcc/testsuite/ChangeLog
> > 2015-10-27 Tsvetkova Alexandra
> >
> > *
Good points.
I will send an updated patch following all your recommendations.
Sebastian
-Original Message-
From: Richard Biener [mailto:richard.guent...@gmail.com]
Sent: Friday, December 11, 2015 3:42 AM
To: Sebastian Pop
Cc: Sebastian Pop; GCC Patches; hiradi...@msn.com
Subject: Re:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68848
Daniel Kahn Gillmor changed:
What|Removed |Added
Attachment #36989|0 |1
is obsolete|
The attached patch enables using r1 to r4 as the literal pool base pointer if
one of them is unused in a leaf function. The unpatched code supports only r5
and r13.
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
gcc/ChangeLog
* config/s390/s390.c (s390_init_frame_layout): Allow
=, arg4=, arg3=,
arg2=, arg1=)
at /usr/local/gcc/gcc-20151211/gcc/testsuite/gcc.dg/guality/pr36728-1.c:12
12char *x = __builtin_alloca (arg7);
$1 =
$2 = 1
!= 1
FAIL: gcc.dg/guality/pr36728-1.c -O3 -g line 16 arg1 == 1
15a1fce36358508909f2013fd6d07e0b9fcad97a is the first bad
Hi,
atm, we dump ssa-name info for lhs-es of statements. That leaves out the
ssa names with default defs.
This proof-of-concept patch prints the ssa-name info for default defs,
in the following format:
...
__attribute__((noclone, noinline))
bar (intD.6 * cD.1755, intD.6 * dD.1756)
# PT =
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=33120
Jan-Benedict Glaw changed:
What|Removed |Added
CC||jbg...@lug-owl.de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68833
--- Comment #3 from Jakub Jelinek ---
Created attachment 37000
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=37000=edit
gcc6-pr68833.patch
Untested fix.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68841
Jakub Jelinek changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68859
Bug ID: 68859
Summary: Add a less strict/smarter version of -Wreorder
Product: gcc
Version: 5.2.0
Status: UNCONFIRMED
Severity: enhancement
Priority: P3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68859
--- Comment #1 from Ryan Johnson ---
(I would be happy to do some legwork on this if somebody is willing to send a
few pointers by PM. I know the code in gcc/cp/init.c, particularly functions
`perform_member_init` and `sort_mem_initializers` are
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=35710
--- Comment #17 from Dominique d'Humieres ---
> Feel free to commit your patch, Dominique. You've been faster with providing
> a fix. :) (But maybe incorporate the whitespace fixes as well.)
Done.
1 - 100 of 271 matches
Mail list logo