Ping.
On Tue, Aug 19, 2014 at 12:45 PM, Kirill Yukhin wrote:
> Hello,
> On 12 Aug 10:58, Andrey Turetskiy wrote:
>> All remarks from
>> https://gcc.gnu.org/ml/gcc-patches/2014-06/msg02166.html are fixed.
>> Updated version of liboffloadmic you can find in GIT branch
>> kyukhin/gomp4-offload:
>> h
Hi all,
Deja pattern in r214086's test is too strict (Asan runtime will not
always be able to detect that invalid memory access is
use-after-poison). I've cooked a trivial patch to loosen the check so
test now also works on i386.
Tested on x64 and i386. Ok to commit?
-Y
commit aa3258be37491
So I should just take that particular check out then?
James Clarke
> On 25 Aug 2014, at 21:05, Mike Stump wrote:
>
>> On Aug 25, 2014, at 11:03 AM, James Clarke wrote:
>> Previously, this macro had taken the form 10X0, where X is the minor
>> version number, e.g. 1090 for OS X 10.9 Mavericks.
Gentlemen!
As discussed behind closed doors previously... I'll be working on
streaming dwarf early. This ultimately benefits Andrew's aspirations
for a gimple type system, removing the front-end tentacles from the rest
of the compilation, and LTO debugging in the presence of language hooks.
On 08/25/2014 03:15 PM, Ville Voutilainen wrote:
Another ICE
Fixed:
Jason
commit 351ac0a147cbfe01e4f8b13a79432d3f37f42caa
Author: Jason Merrill
Date: Mon Aug 25 20:29:21 2014 -0400
* pt.c (check_explicit_specialization): Don't complain about
non-template variable.
(templat
On 08/25/2014 03:15 PM, Ville Voutilainen wrote:
should we just add these test-for-valid cases as new
tests? Or should they become
bug reports instead? Please advise how to best handle these.
Bug reports, please. Or perhaps multiple testcases on one bug report,
for now. :)
Jason
This patch deletes REG_EQUAL note when a src register is replaced by a
constant in an assignment. This is to prevent spurious equivalences
between the constant and the expression in the REG_EQUAL note. In the
bug reported in PR 62146, an assignment in one branch (which is
actually dead) of an IF st
Inside lto-plugin, when $host_subdir is '.', which means GCC is configured
inside source tree, gcc_build_dir is ../../$host_subdir/gcc, not
../$(host_subdir)/gcc. Tested with both in-tree and out-tree build on
Linux/x86-64. OK for trunk?
Thanks.
H.J.
---
2014-08-25 H.J. Lu
PR bootst
Sebastian Pop wrote:
> Richard Biener wrote:
> > I think it would be better to identify a set of features we rely on that
> > are not present in earlier versions and make the test a link
> > test unconditionally.
> >
> > Tobias, are there include files / types / functions we require
> > that are n
On Aug 25, 2014, at 11:03 AM, James Clarke wrote:
> Previously, this macro had taken the form 10X0, where X is the minor
> version number, e.g. 1090 for OS X 10.9 Mavericks. However, as of OS X
> 10.10 Yosemite, it should take the form 10XX00, i.e. 101000 for 10.10.
So, if they continue using the
On Aug 25, 2014, at 12:45 PM, Steven Bosscher wrote:
> On Mon, Aug 25, 2014 at 9:29 PM, Mike Stump wrote:
>> On Aug 25, 2014, at 7:08 AM, David Malcolm wrote:
>>> It's too late now to switch to this approach, so in the meantime I've
>>> been working on ways to make my bootstraps as fast as possibl
On Aug 25, 2014, at 11:03 AM, James Clarke wrote:
> The __DARWIN_64_BIT_INO_T macro is used to indicate whether dirent.d_ino
> is 64-bit or not, and so should be sized appropriately. Additionally,
> dirent.d_seekoff should only be defined/checked when
> __DARWIN_64_BIT_INO_T is defined.
>
> libsa
On Mon, Aug 25, 2014 at 9:29 PM, Mike Stump wrote:
> On Aug 25, 2014, at 7:08 AM, David Malcolm wrote:
>> It's too late now to switch to this approach, so in the meantime I've
>> been working on ways to make my bootstraps as fast as possible.
>
> -j64 works wonders. :-) Though, it is annoying wat
On Aug 25, 2014, at 7:08 AM, David Malcolm wrote:
>
> I'm testing and committing individual patches, but this
> could have gone into trunk in one go.
Yeah, it’s a hard choice between bit rot and incrementally going in.
> what I realize now is that I should have
> continued rebasing my patches i
The following ought to fix two spots where an undefined behavior
can occur when compiling pr28045.c with instrumented compiler.
It does so by changing the type of V to an unsigned HOST_WIDE_INT
and performing the shift on unsigned HOST_WIDE_INT.
Hopefully it doesn't break anything...
Bootstrapped/
Richard Biener wrote:
> I think it would be better to identify a set of features we rely on that
> are not present in earlier versions and make the test a link
> test unconditionally.
>
> Tobias, are there include files / types / functions we require
> that are not available in earlier versions?
On 25 August 2014 17:08, wrote:
>> Anyway, here's a fix for your recent test.
>
> Thanks, I'll take a look at the tests and will send a patch if something
> seems missing; static member variable templates of a class template and
> explicit specializations thereof are the likely starting point.
Please note that this fixes PR 61407 (forgot to add it to
libsanitizer/ChangeLog).
James Clarke
> On 25 Aug 2014, at 19:03, James Clarke wrote:
>
> The __DARWIN_64_BIT_INO_T macro is used to indicate whether dirent.d_ino
> is 64-bit or not, and so should be sized appropriately. Additionally,
>
The __DARWIN_64_BIT_INO_T macro is used to indicate whether dirent.d_ino
is 64-bit or not, and so should be sized appropriately. Additionally,
dirent.d_seekoff should only be defined/checked when
__DARWIN_64_BIT_INO_T is defined.
libsanitizer/ChangeLog:
* sanitizer_common/sanitizer_platform_l
Previously, this macro had taken the form 10X0, where X is the minor
version number, e.g. 1090 for OS X 10.9 Mavericks. However, as of OS X
10.10 Yosemite, it should take the form 10XX00, i.e. 101000 for 10.10.
Added a test for the new format and fixed the formatting for the
existing ones.
gcc/Ch
On 08/21/14 04:30, Richard Biener wrote:
It turns Jeff's jump-threading code in to a strange franken-pass of bits and
pieces of detection and optimisation, and would need some substantial
reworking to fit in with Jeff's changes last Autumn, but if it is more
likely to be acceptable for trunk then
On 08/22/14 06:21, Ilya Enkovich wrote:
Such approach worked well on small tests but trying to run some
benchmarks we faced a problem with reload of address constants. The
problem is that when we try to rematerialize address constant or some
constant memory reference, we have to use pic_offset_
On 08/15/14 18:56, David Malcolm wrote:
On Fri, 2014-08-15 at 18:38 -0400, Trevor Saunders wrote:
i.e. back to just a GET_CODE check again (albeit with a new local, of
the subclass ptr type; I don't know how well our optimizations handle
that yet: the idea that a local ptr X is either equal to a
On 08/15/14 16:38, Trevor Saunders wrote:
On Fri, Aug 15, 2014 at 04:24:49PM -0600, Jeff Law wrote:
On 08/06/14 11:23, David Malcolm wrote:
gcc/
* final.c (get_attr_length_1): Replace GET_CODE check with a
dyn_cast, introducing local "seq" and the use of methods of
rtx_s
This patch is on top of https://gcc.gnu.org/ml/gcc-patches/2014-08/msg02244.html
The main issue here is Wendif-labels, since we were not using the
correct CPP_W_* flag. The rest are straight-forward.
libcpp/ChangeLog:
2014-08-25 Manuel López-Ibáñez
* directives.c (check_eol_1): New.
On Mon, 2014-08-25 18:09:09 +0200, Martin Liška wrote:
> On 08/25/2014 05:29 PM, Jan-Benedict Glaw wrote:
> > Your recent change left one cgraph_state in place in the rs6000
> > backend, resulting in (see
> > http://toolchain.lug-owl.de/buildbot/show_build_details.php?id=346638
> > and
> > http://
On 08/05/2014 10:54 AM, Richard Biener wrote:
The better interface for this is gimple_assign_set_rhs_from_tree. But it
seems the code can be re-structured to avoid re-setting the RHS by
simply doing
if (!is_gimple_reg (x)
&& is_gimple_reg_type (TREE_TYPE (x))
x = init_tm
On 08/22/2014 10:39 PM, Mike Stump wrote:
Before you check it in, consider:
930529-1.x has some alpha bits but I didn’t notice a discussion nor
do I have state on the bug to know if the resolution is correct.
Those are commented out, and I assume superseded by the use of -fwrapv.
Thanks!
Be
> Hello,
>after fixing an issue with callgraph thunk creation, I would like to
> enhance callgraph API a bit. My problem was that I was trying to expand_thunk
> after a body of the original function was removed. As a result, I created a
> call without arguments. The change was suggested by H
On 08/25/2014 05:29 PM, Jan-Benedict Glaw wrote:
Hi!
Your recent change left one cgraph_state in place in the rs6000
backend, resulting in (see
http://toolchain.lug-owl.de/buildbot/show_build_details.php?id=346638
and
http://toolchain.lug-owl.de/buildbot/show_build_details.php?id=346630):
g++
On 08/22/2014 02:14 PM, Joseph S. Myers wrote:
> Tested with no regressions for cross to arm-none-eabi (it also fixes
> failures of gcc.dg/noncompile/920507-1.c, which is PR 61330). OK to
> commit?
>
> 2014-08-22 Joseph Myers
>
> PR target/60606
> PR target/61330
> * varasm.
On Mon, 2014-08-25 11:44:58 -0400, David Edelsohn wrote:
> 2014-08.25 Jan-Benedict Glaw
>
> * config/rs6000/rs6000.c (rs6000_return_in_msb): Fix fallout from
> cgraph_state conversion.
>
>
> The patch seems reasonable to me based on the other changes.
Committed as r214429.
MfG, JBG
--
2014-08.25 Jan-Benedict Glaw
* config/rs6000/rs6000.c (rs6000_return_in_msb): Fix fallout from
cgraph_state conversion.
The patch seems reasonable to me based on the other changes.
Thanks, David
Hi!
Your recent change left one cgraph_state in place in the rs6000
backend, resulting in (see
http://toolchain.lug-owl.de/buildbot/show_build_details.php?id=346638
and
http://toolchain.lug-owl.de/buildbot/show_build_details.php?id=346630):
g++ -c -g -O2 -DIN_GCC -DCROSS_DIRECTORY_STRUCTURE -
On 25 August 2014 16:23:54 CEST, Yury Gribov wrote:
>On 08/25/2014 11:04 AM, Maxim Ostapenko wrote:
>> This patch adds necessary flags to provide a linkage of these tests
>in
> > bootstrap-asan case.
>
>I think you'll want to modify Makefile.def and Makefile.tpl because
>Makefile is generated fro
On 2014-08-22 8:21 AM, Ilya Enkovich wrote:
Hi,
On Cauldron 2014 we had a couple of talks about relaxation of ebx usage in
32bit PIC mode. It was decided that the best approach would be to not fix ebx
register, use speudo register for GOT base address and let allocator do the
rest. This sho
On 08/19/14 15:35, David Malcolm wrote:
On Tue, 2014-08-19 at 13:57 -0700, Richard Henderson wrote:
On 08/06/2014 10:23 AM, David Malcolm wrote:
diff --git a/gcc/cfgrtl.c b/gcc/cfgrtl.c
index 59d633d..5e42a97 100644
--- a/gcc/cfgrtl.c
+++ b/gcc/cfgrtl.c
@@ -1604,6 +1604,7 @@ force_nonfallthru_a
On 08/25/2014 11:04 AM, Maxim Ostapenko wrote:
> This patch adds necessary flags to provide a linkage of these tests in
> bootstrap-asan case.
I think you'll want to modify Makefile.def and Makefile.tpl because
Makefile is generated from them.
-Y
On 08/19/14 14:41, Richard Henderson wrote:
On 08/06/2014 10:23 AM, David Malcolm wrote:
gcc/
* rtl.h (rtx_expr_list::insn): New method.
---
gcc/rtl.h | 9 +
1 file changed, 9 insertions(+)
diff --git a/gcc/rtl.h b/gcc/rtl.h
index d028be1..d5811c2 100644
--- a/gcc/rtl.h
+++ b
On Mon Aug 25 2014 15:47:26 GMT+0300 (EEST), Jason Merrill wrote:
> On 08/25/2014 05:24 AM, Ville Voutilainen wrote:
> > Well, it's a bit premature to add tests when the implementation itself is
> > not
> > quite complete, I guess. :) Having ICEs makes it hard to add tests, too. :P
>
> But test
On Sat, 2014-08-23 at 20:49 +0200, Jan-Benedict Glaw wrote:
> On Wed, 2014-08-06 13:19:48 -0400, David Malcolm wrote:
> > This is further scaffolding; convert the BB_* and SET_BB_* macros
> > into functions. Convert the BB_* rvalue-style functions into returning
> > rtx_insn * rather than plain r
Another issue found by -Wlogical-not-parentheses.
tree-vect-data-refs.c contains
if (!LOOP_REQUIRES_VERSIONING_FOR_ALIGNMENT (loop_vinfo))
but since the definition of LOOP_REQUIRES_VERSIONING_FOR_ALIGNMENT
is lacking parens, this expands to !X > 0 and we warn.
Similarly for LOOP_REQUIRES_VERSIONING
On 08/25/2014 05:24 AM, Ville Voutilainen wrote:
Well, it's a bit premature to add tests when the implementation itself is not
quite complete, I guess. :) Having ICEs makes it hard to add tests, too. :P
But tests are how you verify that the implementation is complete.
Anyway, here's a fix for
On Mon, Aug 25, 2014 at 2:24 PM, Marek Polacek wrote:
>> Done.
>>
>> 2014-08-25 Petr Murzin
>>
>> * config/i386/i386.md (SWI1248_AVX512BW): New mode iterator.
>> (*k): Add *kqi and *khi and use
>> SWI1248_AVX512BW mode iterator.
>
> This broke gcc build:
> i386.md:894: unterminated construct
>
On Mon, Aug 25, 2014 at 02:52:16PM +0400, Petr Murzin wrote:
> Done.
>
> 2014-08-25 Petr Murzin
>
> * config/i386/i386.md (SWI1248_AVX512BW): New mode iterator.
> (*k): Add *kqi and *khi and use
> SWI1248_AVX512BW mode iterator.
This broke gcc build:
i386.md:894: unterminated construct
Uros,
2014-08-22 16:36 GMT+04:00 Martin Liška :
> On 08/13/2014 02:25 PM, Ilya Enkovich wrote:
>> Hi,
>>
>> This patch is to fix wrong refactoring for cgraph_node::function_symbol
>> introduced by this patch:
>> https://gcc.gnu.org/ml/gcc-cvs/2014-07/msg00805.html. Here is how function
>> was refacto
2014-08-25 15:24 GMT+04:00 Hans-Peter Nilsson :
> On Mon, 25 Aug 2014, Ilya Enkovich wrote:
>> 2014-08-23 5:47 GMT+04:00 Hans-Peter Nilsson :
>> > ...did you send the right version of the patch?
>> > This one uses the RTX-returning hook only in boolean tests,
>> > unless I misread.
>
> (I did, but
PR62024 reports that we can't use __atomic_always_lock_free in
a static assert, as the FEs say it's not a constant expression. Yet the
docs say that the result of __atomic_always_lock_free is a compile time
constant.
We can fix this pretty easily. While fold folds __atomic_always_lock_free
to a c
On Mon, 25 Aug 2014, Ilya Enkovich wrote:
> 2014-08-23 5:47 GMT+04:00 Hans-Peter Nilsson :
> > ...did you send the right version of the patch?
> > This one uses the RTX-returning hook only in boolean tests,
> > unless I misread.
(I did, but not by much.)
> NULL returned by hook means we do not ha
Done.
2014-08-25 Petr Murzin
* config/i386/i386.md (SWI1248_AVX512BW): New mode iterator.
(*k): Add *kqi and *khi and use
SWI1248_AVX512BW mode iterator.
Best regards,
Petr Murzin
On Mon, Aug 25, 2014 at 1:42 PM, Uros Bizjak wrote:
> On Mon, Aug 25, 2014 at 11:00 AM, Petr Murzin wrote:
>
>
Hi,
the S/390 backend splits RTXs like (set (reg:DI 3) (neg:DI (reg:DI
2))) on 31bit ESA in a way which overwrites r3 before it is read as
source operand.
The attached patch adds a check to the insn condition and provides a
second splitter as plan B.
This fixes ESA mode bootstrap on s390. No reg
Hi all,
The bug is reported at https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56846, and
it’s about the problem that
when exception handler is involved in the function, then _Unwind_Backtrace
function will run into deadloop on
arm target.
Cmd line: arm-none-eabi-g++ -mthumb -mcpu=cortex-m3 -O0 -
The attached patch fixes a bessel_7 fortran test failure that appeared during
SLES 12 distro tests for s390x - by bumping the error tolerance.
The same patch should apply to gcc-4.9 and gcc-4.8.
ChangeLog:
--
2014-08-25 Dominik Vogt
* gfortran.dg/bessel_7.f90: Bump allowed precision t
Hello,
after fixing an issue with callgraph thunk creation, I would like to enhance
callgraph API a bit. My problem was that I was trying to expand_thunk after a
body of the original function was removed. As a result, I created a call
without arguments. The change was suggested by Honza.
Th
On Mon, Aug 25, 2014 at 11:00 AM, Petr Murzin wrote:
> I've macroized logic patterns. Please have a look. Is it ok for trunk?
>
> 2014-08-25 Petr Murzin
>
> * config/i386/i386.md: Macroize logic patterns.
Please write ChangeLog entry like:
*config/i386/i386.md (SWI1248_AVX512BW): New mode it
Hi,
I'm going to commit as obvious the below when testing finishes.
Thanks,
Paolo.
///
2014-08-25 Paolo Carlini
* g++.old-deja/g++.pt/var1.C: Use c++14, not c++1y, in dg-error.
Index: g++.old-deja/g++.pt/var1.C
2014-08-23 5:47 GMT+04:00 Hans-Peter Nilsson :
> (Dropping gcc@ and people known to subscribe to gcc-patches
> from the CC.)
>
> Sorry for the drive-by review, but...
>
> On Fri, 22 Aug 2014, Ilya Enkovich wrote:
>> Hi,
>>
>> On Cauldron 2014 we had a couple of talks about relaxation of
>> ebx usag
On 25 August 2014 05:54, Jason Merrill wrote:
> On 08/23/2014 06:29 PM, Ville Voutilainen wrote:
>>
>> Based on my quick tests with it, it seems to me that we need more
>> tests for static member variable templates. My first attempt at something
>> like that gave an ICE...
>
>
> Yeah, I figured th
On Mon, Aug 25, 2014 at 09:19:08AM +0200, Gerald Pfeifer wrote:
> On Fri, 22 Aug 2014, Marek Polacek wrote:
> Nice! Thanks you.
Thanks for review.
> Oh, yes, okay. Can you just check that lines are <80 columns?
Sure. (I actually had to re-indent a few of them.)
Committed.
Marek
Hi,
I've macroized logic patterns. Please have a look. Is it ok for trunk?
2014-08-25 Petr Murzin
* config/i386/i386.md: Macroize logic patterns.
logic_patterns_patch
Description: Binary data
On Sat, 23 Aug 2014, Raghunath Lolur wrote:
> Please find an update of test results for 4.8.x
>
> Test Results for 4.8.3:
> aarch64-unknown-linux-gnu
Applied, thanks.
Gerald
On Fri, 22 Aug 2014, Marek Polacek wrote:
> Elsewhere in the code base I see mainly "pointer to integer" so I
> haven't changed this.
Sounds good.
> The following patch contains two more options: -Wc99-c11-compat and
> -Wbool-compare.
>
> Ok?
Nice! Thanks you.
Oh, yes, okay. Can you just c
Hi,
When I ran Asan tests under Asan-bootstrapped GCC 5.0, I've noted, that
tests for libiberty and libbacktrace fail to link with sanitized
libbacktrace.a and libiberty.a because of missing -static-libasan
-fsanitize=address linker flags.
This patch adds necessary flags to provide a linkage
63 matches
Mail list logo
