--- Comment #23 from jakub at gcc dot gnu dot org 2010-06-25 12:30 ---
Fixed on the trunk so far.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Known
--- Comment #1 from jakub at gcc dot gnu dot org 2010-06-25 13:10 ---
For DW_AT_low_pc, we do this because of (DWARF4, 3.1.1:
A DW_AT_low_pc attribute may also be specified in combination with
DW_AT_ranges to specify the default base address for use in location lists (see
Section 2.6.2
-optimization
Severity: normal
Priority: P3
Component: tree-optimization
AssignedTo: unassigned at gcc dot gnu dot org
ReportedBy: jakub at gcc dot gnu dot org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44669
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Target Milestone|--- |4.6.0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44669
--- Comment #2 from jakub at gcc dot gnu dot org 2010-06-25 16:53 ---
Created an attachment (id=21005)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21005action=view)
gcc46-pr44668.patch
Patch to fix this for typedefs. More work will be needed to handle it for
other DW_TAG_* type
--- Comment #15 from jakub at gcc dot gnu dot org 2010-06-25 18:11 ---
Well, the compiler shouldn't ICE on invalid input.
And, delta is just a simple text tool, it depends on the user how he writes the
test script. Ideally when reducing a testcase to find an ICE the delta test
script
--- Comment #3 from jakub at gcc dot gnu dot org 2010-06-25 18:23 ---
Created an attachment (id=21006)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21006action=view)
gcc46-pr44668.patch
Updated patch.
--
jakub at gcc dot gnu dot org changed:
What|Removed
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned at gcc dot gnu |jakub at gcc dot gnu dot org
|dot org
--- Comment #3 from jakub at gcc dot gnu dot org 2010-06-24 09:55 ---
Created an attachment (id=20994)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20994action=view)
gcc46-pr44539.patch
Untested fix.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44539
--- Comment #5 from jakub at gcc dot gnu dot org 2010-06-24 10:44 ---
Created an attachment (id=20996)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20996action=view)
gcc46-pr44539.patch
This works too.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44539
--- Comment #8 from jakub at gcc dot gnu dot org 2010-06-24 10:57 ---
The cas/casx insns only allow (mem (reg)) addressing:
(match_operand:I48MODE 1 memory_reg_operand +m)
(match_operand:DI 1 memory_reg_operand +m)
The memory_reg_operand predicate checks this and fails
--- Comment #4 from jakub at gcc dot gnu dot org 2010-06-24 11:00 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCONFIRMED
--- Comment #5 from jakub at gcc dot gnu dot org 2010-06-24 11:24 ---
Is this now fixed?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44203
--- Comment #5 from jakub at gcc dot gnu dot org 2010-06-24 11:25 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCONFIRMED
--- Comment #8 from jakub at gcc dot gnu dot org 2010-06-24 11:33 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #4 from jakub at gcc dot gnu dot org 2010-06-24 11:51 ---
note-type == 10 is:
*d = '\n';
/* A sentinel note that should never be processed. */
add_line_note (buffer, d + 1, '\n');
buffer-next_line = s + 1;
created. But, as this PR lacks a testcase (well
--- Comment #8 from jakub at gcc dot gnu dot org 2010-06-24 11:58 ---
Fixed on the trunk.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Summary
--- Comment #6 from jakub at gcc dot gnu dot org 2010-06-24 12:56 ---
Can't reproduce on x86_64-linux (and, #pragma redefine_extname seems to be
handled on all targets, not just Solaris).
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39213
--- Comment #10 from jakub at gcc dot gnu dot org 2010-06-24 13:31 ---
This looks like a bug in the source to me, certainly not a middle-end or
tree-optimization bug.
The
a_sp = matrix%local_data_sp
assignment when
matrix%local_data_sp
is uninitialized means you are copying
--- Comment #11 from jakub at gcc dot gnu dot org 2010-06-24 16:21 ---
Created an attachment (id=20997)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20997action=view)
unswitch-tuplification.patch
While the validity of the testcase is questionable, what unswitching does looks
--- Comment #12 from jakub at gcc dot gnu dot org 2010-06-24 16:24 ---
Created an attachment (id=20998)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20998action=view)
level-opt.patch
When we reach the max-unswitch-level param, the function returns immediately,
which means
--- Comment #13 from jakub at gcc dot gnu dot org 2010-06-24 16:47 ---
For the last issue, perhaps doing delete_unreachable_blocks in between the
levels wouldn't work too well, loops would need to be discovered again etc.
Maybe we can just do something similar to find_unreachable_blocks
--- Comment #23 from jakub at gcc dot gnu dot org 2010-06-24 17:48 ---
Subject: Bug 44492
Author: jakub
Date: Thu Jun 24 17:48:16 2010
New Revision: 161328
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161328
Log:
PR middle-end/44492
* recog.h (struct recog_data
--- Comment #6 from jakub at gcc dot gnu dot org 2010-06-22 05:50 ---
Why not just:
TARGET_EITHER (arm_arch6 || !optimize_size)
? That !arm_arch6 is completely redundant there.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44392
--- Comment #2 from jakub at gcc dot gnu dot org 2010-06-22 09:33 ---
This is a bug in the testcase.
The asms don't have memory clobber, so memory stores can be scheduled across
it. If you add it, the testcase works fine.
--
jakub at gcc dot gnu dot org changed:
What
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned at gcc dot gnu |jakub at gcc dot gnu dot org
|dot org
--- Comment #1 from jakub at gcc dot gnu dot org 2010-06-22 09:58 ---
Created an attachment (id=20977)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20977action=view)
gcc46-pr44627.patch
Untested fix.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44627
--- Comment #2 from jakub at gcc dot gnu dot org 2010-06-22 10:32 ---
Created an attachment (id=20978)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20978action=view)
gcc46-pr44619.patch
Untested fix.
--
jakub at gcc dot gnu dot org changed:
What|Removed
--- Comment #3 from jakub at gcc dot gnu dot org 2010-06-22 11:37 ---
It does, you need latest trunk though (r161079 or later).
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44612
--- Comment #7 from jakub at gcc dot gnu dot org 2010-06-22 14:12 ---
Well, RTL DSE to some extent knows about memset.
And, for tree DSE handling it would be much easier and desirable.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44612
--- Comment #13 from jakub at gcc dot gnu dot org 2010-06-22 15:25 ---
Well, a(5:)=0.0 doesn't satisfy copyable_array_p, so gfc_trans_zero_assign
isn't called at all.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41137
--- Comment #3 from jakub at gcc dot gnu dot org 2010-06-22 16:46 ---
3.4 gives here:
pr44629.C:23: sorry, unimplemented: use of `overload' in template type
unification
pr44629.C:23: error: no matching function for call to
`Grapher::simple_output(Searcherchar, unsafe_case_cmp [with T
--- Comment #2 from jakub at gcc dot gnu dot org 2010-06-22 16:58 ---
Well, -i4 (and -i8) actually has never been supported by gfortran it seems.
Just with gfortran 4.3 and earlier you'd get errors only when compiling
files with *.F/*.F90/*.F03 etc. extensions (i.e. when separate
--- Comment #3 from jakub at gcc dot gnu dot org 2010-06-22 17:13 ---
Well, for -O1 the ullong - double conversion is done at compile time instead
of runtime.
Does the problem occur also when l is long long instead of unsigned long long?
Can you check what value the fxtod insn computes
--- Comment #2 from jakub at gcc dot gnu dot org 2010-06-22 20:43 ---
Subject: Bug 44627
Author: jakub
Date: Tue Jun 22 20:42:50 2010
New Revision: 161227
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161227
Log:
PR c++/44627
* error.c (dump_expr): Don't look
--- Comment #3 from jakub at gcc dot gnu dot org 2010-06-22 20:47 ---
Subject: Bug 44627
Author: jakub
Date: Tue Jun 22 20:46:51 2010
New Revision: 161228
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161228
Log:
PR c++/44627
* error.c (dump_expr): Don't look
--- Comment #4 from jakub at gcc dot gnu dot org 2010-06-22 20:48 ---
Subject: Bug 44627
Author: jakub
Date: Tue Jun 22 20:48:24 2010
New Revision: 161229
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161229
Log:
PR c++/44627
* error.c (dump_expr): Don't look
--- Comment #3 from jakub at gcc dot gnu dot org 2010-06-22 20:50 ---
Subject: Bug 44619
Author: jakub
Date: Tue Jun 22 20:50:03 2010
New Revision: 161230
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161230
Log:
PR c++/44619
* typeck2.c (build_m_component_ref
--- Comment #13 from jakub at gcc dot gnu dot org 2010-06-22 20:56 ---
It looks much better than adding new single register constraints to me.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44618
--- Comment #4 from jakub at gcc dot gnu dot org 2010-06-22 21:14 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #5 from jakub at gcc dot gnu dot org 2010-06-22 21:14 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #4 from jakub at gcc dot gnu dot org 2010-06-21 06:11 ---
FYI, this has nothing to do with the new -Wunused-but-set-* warnings, the
initialization in the definition isn't counted as set.
This is something the old -Wunused-variable warning reports/doesn't report.
--
http
--- Comment #3 from jakub at gcc dot gnu dot org 2010-06-21 12:49 ---
Created an attachment (id=20960)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20960action=view)
gcc46-pr44575.patch
Untested fix.
--
jakub at gcc dot gnu dot org changed:
What|Removed
--- Comment #14 from jakub at gcc dot gnu dot org 2010-06-21 16:27 ---
Subject: Bug 44426
Author: jakub
Date: Mon Jun 21 16:26:25 2010
New Revision: 161092
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161092
Log:
PR bootstrap/44426
* sel-sched-dump.h
--- Comment #4 from jakub at gcc dot gnu dot org 2010-06-21 16:34 ---
Subject: Bug 44575
Author: jakub
Date: Mon Jun 21 16:33:49 2010
New Revision: 161097
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161097
Log:
PR target/44575
* config/i386/i386.c
--- Comment #15 from jakub at gcc dot gnu dot org 2010-06-21 17:07 ---
Subject: Bug 44426
Author: jakub
Date: Mon Jun 21 17:06:48 2010
New Revision: 161100
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161100
Log:
PR bootstrap/44426
* sel-sched-dump.h
--- Comment #16 from jakub at gcc dot gnu dot org 2010-06-21 17:11 ---
Subject: Bug 44426
Author: jakub
Date: Mon Jun 21 17:10:02 2010
New Revision: 161102
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161102
Log:
PR bootstrap/44426
* sel-sched-dump.h
--- Comment #17 from jakub at gcc dot gnu dot org 2010-06-21 17:20 ---
Should be fixed now.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #11 from jakub at gcc dot gnu dot org 2010-06-21 17:43 ---
What's the reason why gfc_trans_zero_assign insists that len is INTEGER_CST?
At least if it is contiguous (and not assumed size), why can't memset be used
even for non-constant sizes?
--
http://gcc.gnu.org
--- Comment #1 from jakub at gcc dot gnu dot org 2010-06-18 07:04 ---
Regressed with r146817 (SSA expand).
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44575
Severity: normal
Priority: P3
Component: c++
AssignedTo: unassigned at gcc dot gnu dot org
ReportedBy: jakub at gcc dot gnu dot org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44587
--- Comment #1 from jakub at gcc dot gnu dot org 2010-06-18 21:54 ---
Compiled fine until r144617, starting with r144618 gave
R.ii:2: error: âBT::câ cannot appear in a constant-expression
R.ii:2: error: template argument 1 is invalid
and starting with r154042 started ICEing
--- Comment #4 from jakub at gcc dot gnu dot org 2010-06-17 12:28 ---
Created an attachment (id=20932)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20932action=view)
callgrind.out.bz2
Seems it is mainly df and RA that eats the time for -O0 -g0.
--
http://gcc.gnu.org/bugzilla
--- Comment #1 from jakub at gcc dot gnu dot org 2010-06-17 19:16 ---
Subject: Bug 44572
Author: jakub
Date: Thu Jun 17 19:15:51 2010
New Revision: 160944
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=160944
Log:
PR debug/44572
* dwarf2out.c
--- Comment #2 from jakub at gcc dot gnu dot org 2010-06-17 19:18 ---
Hopefully fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status
--- Comment #13 from jakub at gcc dot gnu dot org 2010-06-16 06:55 ---
This looks wrong. The code in expand_one_stack_var_at (before dynamic stack
realignment) made perfect sense, if we gave a bigger alignment to some variable
(e.g. automatic array), it is useful to tell the expanders
--- Comment #5 from jakub at gcc dot gnu dot org 2010-06-16 09:45 ---
The __cancel_arg variables are pseudos until ira, apparently during IRA the 2
stack slots chosen for those are shared between __cancel_arg vars and other
vars used later in the function. Do we need to act as if
-fno
--- Comment #15 from jakub at gcc dot gnu dot org 2010-06-16 14:57 ---
1) all the world isn't ix86/x86_64, this is generic code, so
MAX_SUPPORTED_STACK_ALIGNMENT is small on many targets
2) even when get_decl_align_unit returns something small, the decl might still
get a nicely aligned
Component: target
AssignedTo: unassigned at gcc dot gnu dot org
ReportedBy: jakub at gcc dot gnu dot org
GCC target triplet: x86_64-linux
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44542
--- Comment #2 from jakub at gcc dot gnu dot org 2010-06-15 11:36 ---
I don't have any wrong-code testcases.
__attribute__((noinline, noclone)) void
f (long x)
{
long a, b, c, d;
asm ();
__builtin_alloca (1);
}
int
main (void)
{
f (1234567890);
return 0;
}
shows the same
--- Comment #1 from jakub at gcc dot gnu dot org 2010-06-15 12:15 ---
Subject: Bug 44536
Author: jakub
Date: Tue Jun 15 12:15:26 2010
New Revision: 160779
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=160779
Log:
PR fortran/44536
* langhooks.h (struct
--- Comment #2 from jakub at gcc dot gnu dot org 2010-06-15 12:17 ---
Subject: Bug 44536
Author: jakub
Date: Tue Jun 15 12:17:16 2010
New Revision: 160780
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=160780
Log:
PR fortran/44536
* langhooks.h (struct
--- Comment #3 from jakub at gcc dot gnu dot org 2010-06-15 12:27 ---
Subject: Bug 44536
Author: jakub
Date: Tue Jun 15 12:27:01 2010
New Revision: 160783
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=160783
Log:
PR fortran/44536
* langhooks.h (struct
--- Comment #4 from jakub at gcc dot gnu dot org 2010-06-15 12:49 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCONFIRMED
--- Comment #3 from jakub at gcc dot gnu dot org 2010-06-15 12:53 ---
Created an attachment (id=20914)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20914action=view)
gcc46-pr44542.patch
For the don't use changed DECL_ALIGN for stack estimation after
expand_one_stack_var_at has
--- Comment #1 from jakub at gcc dot gnu dot org 2010-06-15 16:32 ---
Subject: Bug 44544
Author: jakub
Date: Tue Jun 15 16:32:24 2010
New Revision: 160798
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=160798
Log:
PR bootstrap/44544
Backport from mainline
--- Comment #9 from jakub at gcc dot gnu dot org 2010-06-15 16:45 ---
Re: #c4 - !really_expand never occur when !optimize and for optimize they
are called IMHO way too early (during inlining etc.).
Re: #c8 - the testcases were meant for the given range of svn revs of trunk to
show
--- Comment #2 from jakub at gcc dot gnu dot org 2010-06-15 16:57 ---
Should be fixed now, sorry.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #3 from jakub at gcc dot gnu dot org 2010-06-15 17:12 ---
Caused by http://gcc.gnu.org/viewcvs?root=gccview=revrev=159362
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #17 from jakub at gcc dot gnu dot org 2010-06-14 06:59 ---
Created an attachment (id=20906)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20906action=view)
gcc46-pr44509.patch
Untested patch.
--
jakub at gcc dot gnu dot org changed:
What|Removed
--- Comment #1 from jakub at gcc dot gnu dot org 2010-06-14 08:01 ---
tempreg = convert_to_mode (data-nominal_mode, tempreg, unsignedp);
if (GET_CODE (tempreg) == SUBREG
GET_MODE (tempreg) == data-nominal_mode
REG_P (SUBREG_REG (tempreg
--- Comment #16 from jakub at gcc dot gnu dot org 2010-06-14 08:43 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #10 from jakub at gcc dot gnu dot org 2010-06-14 08:54 ---
In sel_print* case I wonder why it is defined as macros at all, it surely
bloats sel-sched* a lot for something that isn't enabled by default (verbose
dumps).
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44426
--- Comment #11 from jakub at gcc dot gnu dot org 2010-06-14 10:20 ---
Patch for sel_print posted:
http://gcc.gnu.org/ml/gcc-patches/2010-06/msg01400.html
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44426
--- Comment #19 from jakub at gcc dot gnu dot org 2010-06-14 12:31 ---
Subject: Bug 44509
Author: jakub
Date: Mon Jun 14 12:30:29 2010
New Revision: 160729
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=160729
Log:
PR bootstrap/44509
* c-config-lang.in (gtfiles
--- Comment #21 from jakub at gcc dot gnu dot org 2010-06-14 12:32 ---
Fixed, sorry for the breakage.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #2 from jakub at gcc dot gnu dot org 2010-06-14 15:19 ---
Subject: Bug 44508
Author: jakub
Date: Mon Jun 14 15:19:04 2010
New Revision: 160749
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=160749
Log:
PR tree-optimization/44508
* tree-ssa-propagate.h
--- Comment #3 from jakub at gcc dot gnu dot org 2010-06-14 15:24 ---
Subject: Bug 44508
Author: jakub
Date: Mon Jun 14 15:23:31 2010
New Revision: 160750
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=160750
Log:
PR tree-optimization/44508
* tree-ssa-propagate.h
--- Comment #4 from jakub at gcc dot gnu dot org 2010-06-14 15:28 ---
Subject: Bug 44508
Author: jakub
Date: Mon Jun 14 15:28:00 2010
New Revision: 160752
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=160752
Log:
PR tree-optimization/44508
* tree-ssa-propagate.c
--- Comment #5 from jakub at gcc dot gnu dot org 2010-06-14 15:53 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|NEW
--- Comment #12 from jakub at gcc dot gnu dot org 2010-06-14 15:54 ---
Subject: Bug 44426
Author: jakub
Date: Mon Jun 14 15:53:38 2010
New Revision: 160754
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=160754
Log:
PR bootstrap/44426
* tree.h (build_call_expr
--- Comment #13 from jakub at gcc dot gnu dot org 2010-06-14 16:01 ---
Subject: Bug 44426
Author: jakub
Date: Mon Jun 14 16:00:39 2010
New Revision: 160755
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=160755
Log:
PR bootstrap/44426
* tree.h (build_call_expr
Priority: P3
Component: tree-optimization
AssignedTo: unassigned at gcc dot gnu dot org
ReportedBy: jakub at gcc dot gnu dot org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44508
--- Comment #2 from jakub at gcc dot gnu dot org 2010-06-12 11:51 ---
Can't reproduce with
double a = __DBL_MAX__ + __FLT_MAX__;
in x86_64-linux - x86_64-apple-darwin10 cross (not even when using precompiled
header).
So, please provide preprocessed testcase, and if possible in gdb also
--- Comment #5 from jakub at gcc dot gnu dot org 2010-06-12 13:37 ---
Can't reproduce with cross even on that preprocessed source.
Just run
gdb ./cc1 -O2 pr44509.i
and in gdb type:
r
up
p *node
p *node-value.macro
p node-value.macro-exp.tokens[0]
p node-value.macro-exp.tokens[1
--- Comment #12 from jakub at gcc dot gnu dot org 2010-06-12 19:29 ---
Instead of spending too much time on testsuite reduction, perhaps if you could
put a breakpoint on builtin_define_with_hex_fp_value where
lazy_hex_fp_values[XXX].macro is initialized (for XXX being the idx which
--- Comment #13 from jakub at gcc dot gnu dot org 2010-06-12 19:41 ---
Argh, I thought macro is allocated using _cpp_aligned_alloc, but apparently it
is allocated using GC:
if (pfile-hash_table-alloc_subobject)
macro = (cpp_macro *) pfile-hash_table-alloc_subobject
(sizeof
--- Comment #22 from jakub at gcc dot gnu dot org 2010-06-11 13:22 ---
Created an attachment (id=20888)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20888action=view)
gcc46-pr44492.patch
Updated patch.
--
jakub at gcc dot gnu dot org changed:
What|Removed
operands
Product: gcc
Version: 4.6.0
Status: UNCONFIRMED
Keywords: wrong-code
Severity: normal
Priority: P3
Component: middle-end
AssignedTo: unassigned at gcc dot gnu dot org
ReportedBy: jakub at gcc dot gnu dot org
--- Comment #2 from jakub at gcc dot gnu dot org 2010-06-10 09:44 ---
struct T { unsigned long p; };
struct S { T a, b, c; unsigned d; };
__attribute__((noinline))
void
bar (const T x, const T y)
{
if (x.p != 0x2348 || y.p != 0x2346)
__builtin_abort ();
}
__attribute__((noinline
--- Comment #5 from jakub at gcc dot gnu dot org 2010-06-10 10:25 ---
And ensure that it is used exactly once in an inline asm pattern? Even for g
constraint? That just can't be true.
Just look at how many of m and =m constraints e.g. glibc uses.
Only 8 occurrences of %U, but e.g
--- Comment #8 from jakub at gcc dot gnu dot org 2010-06-10 11:36 ---
Don't understand the last comment.
If matched constraint can't have side-effects, then this bug is valid and
auto-inc-dec.c should just do if (asm_noperands (insn) = 0) continue;
similarly how it refuses to auto-inc
--- Comment #11 from jakub at gcc dot gnu dot org 2010-06-10 12:24 ---
I believe for GCC it shouldn't be hard to at least easily detect the used zero
times case which happens very often in lost of code.
asm (... %2 ... : =m (*p) : m (*p), r (p));
is just very common, the =m and m just
--- Comment #13 from jakub at gcc dot gnu dot org 2010-06-10 15:16 ---
Subject: Bug 43838
Author: jakub
Date: Thu Jun 10 15:15:18 2010
New Revision: 160554
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=160554
Log:
PR other/43838
* cp-demangle.c (struct
--- Comment #20 from jakub at gcc dot gnu dot org 2010-06-10 15:22 ---
Another option is to disallow side-effects in inline asm memory operands and
let the user explicitly request that the side-effects are possible, through
some new constraint modifier (e.g. _ used as _mi or =_g would
--- Comment #14 from jakub at gcc dot gnu dot org 2010-06-10 15:24 ---
Subject: Bug 43838
Author: jakub
Date: Thu Jun 10 15:24:11 2010
New Revision: 160555
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=160555
Log:
PR other/43838
* cp-demangle.c (struct
--- Comment #8 from jakub at gcc dot gnu dot org 2010-06-10 15:32 ---
Subject: Bug 10197
Author: jakub
Date: Thu Jun 10 15:31:56 2010
New Revision: 160556
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=160556
Log:
PR other/43838
* cp-demangle.c (struct
--- Comment #15 from jakub at gcc dot gnu dot org 2010-06-10 15:32 ---
Subject: Bug 43838
Author: jakub
Date: Thu Jun 10 15:31:56 2010
New Revision: 160556
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=160556
Log:
PR other/43838
* cp-demangle.c (struct
--- Comment #21 from jakub at gcc dot gnu dot org 2010-06-10 17:06 ---
Created an attachment (id=20883)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20883action=view)
gcc46-pr44494.patch
WIP patch that implements that. Still missing documentation, testcases and
likely also doing
--- Comment #19 from jakub at gcc dot gnu dot org 2010-06-09 09:39 ---
Created an attachment (id=20874)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20874action=view)
CVE-2010-0831.patch
Just for the record, the patch that went in leaves fastjar still vulnerable.
The main issue
501 - 600 of 6780 matches
Mail list logo