[patch testsuite]: Fixes in gcc.dg for mingw-targets

2013-03-07 Thread Kai Tietz
Hi, this patch fixes some remaining issue for mingw-targets in gcc.dg's testsuite. ChangeLog 2013-03-07 Kai Tietz kti...@redhat.com * gcc.dg/pr14092-1.c: Mark intptr_t typedef to use extension. * gcc.dg/pr24683.c: Avoid warning about casting constant string. * gcc.dg

[patch testsuite]: Fix regressions in gcc-c-torture's builtin-tests using LTO on mingw-targets

2013-03-08 Thread Kai Tietz
Hello, this patch fixes 23 regressions about LTO and double-defined symbols for memcpy/memset. Issue is that for mingw-targets the C-runtime functions aren't weak and not necessarily overridable. ChangeLog 2013-03-08 Kai Tietz kti...@redhat.com * gcc.c-torture/execute/builtins

Re: [patch testsuite]: Fix regressions in gcc-c-torture's builtin-tests using LTO on mingw-targets

2013-03-08 Thread Kai Tietz
2013/3/8 Rainer Orth r...@cebitec.uni-bielefeld.de: Hi Kai, Index: builtins.exp === --- builtins.exp(Revision 196543) +++ builtins.exp(Arbeitskopie) @@ -42,6 +42,7 @@ if [istarget powerpc-*-darwin*] {

Re: [patch testsuite]: Fix gcc.target/i386 cases for mingw-targets

2013-03-08 Thread Kai Tietz
2013/3/8 Rainer Orth r...@cebitec.uni-bielefeld.de: Hi Kai, some comments; I'll leave approval to a target maintainer. Index: gcc.target/i386/movti.c === --- gcc.target/i386/movti.c (Revision 196507) +++

Re: [patch testsuite]: Fix gcc.target/i386 cases for mingw-targets

2013-03-12 Thread Kai Tietz
What's here to ping about? I got ok by rth. Kai

[patch libffi]: Make sure code is position-independent for x64 targets

2013-03-22 Thread Kai Tietz
Hi, this patch fixes the Win64-code so that we use only pc-relative addressing. This is of importance if code gets linked to an pe-image with an image-base above 2GB. ChangeLog 2013-03-22 Kai Tietz kti...@redhat.com * src/x86/win64.S: Make use of ffi_closure_win64_inner

[patch i386]: Treat dllimported symbol as external in predicates

2013-03-22 Thread Kai Tietz
Hi, A dllimported symbol is always external. So treat that proper in local_symbolic_operand. ChangeLog 2013-03-22 Kai Tietz kti...@redhat.com * config/i386/predicates.md (local_symbolic_operand): Interprete dll-imported symbols as none-local. Tested for x86_64-w64-mingw32

[patch mingw]: Make POSIX-printf for mingw-based builds default

2013-03-22 Thread Kai Tietz
Hi, this patch enables the POSIX-printf variant for mingw-hosts by default. ChangeLog 2013-03-22 Kai Tietz kti...@redhat.com * config/i386/xm-mingw32.h (__USE_MINGW_ANSI_STDIO): Enable POSIX-printf for mingw-hosted builds. Tested for i686-w64-mingw32, and x86_64-w64-mingw32

[patch java]: Avoid looping over the end_params_node in put_decl_node

2013-03-22 Thread Kai Tietz
Hi, this patch makes sure we don't walk in put_decl_node of the end_params_node element. ChangeLog 2013-03-22 Kai Tietz kti...@redhat.com * lang.c (put_decl_node): Don't iterate over end_params_node. Tested for i686-w64-mingw32, x86_64-w64-mingw32, and x86_64-unknown-linux-gnu. Ok

[patch testsuite]: g++.dg

2013-03-22 Thread Kai Tietz
Hi, this patch fixes an LLP64 issue in g++.dg's testsuite. ChangeLog 2013-03-22 Kai Tietz kti...@redhat.com * g++.dg/torture/20121105-1.C: Adjust for LLP64 targets. Ok for apply? Regards, Kai Index: gcc/testsuite/g++.dg/torture/20121105-1.C

[patch libstdc++ testsuite]: Add new option dg-additional-options and make use of it for mingw targets

2013-03-22 Thread Kai Tietz
within DLL. ChangeLog 2013-03-22 Kai Tietz kti...@redhat.com * 18_support/50594.cc: For mingw-targets use only static libstdc++-version. * 19_diagnostics/error_category/operators/equal.cc * 19_diagnostics/error_code/cons/1.cc * 19_diagnostics/error_code

[patch libgcc]: Prepare cygwin.S file to share code with cygwin64

2013-03-22 Thread Kai Tietz
Hi, this patch replaces use of _WIN64 by __x86_64__ so mingw x64 and cygwin x64 version can share same source. ChangeLog 2013-03-22 Kai Tietz kti...@redhat.com * config/i386/cygwin.S: Replace use of _WIN64 by __x86_64__. Tested for x86_64-w64-mingw32, x86_64-pc-cygwin, and i686-w64

[patch libgcc]: Adjust cygming-crtbegin code to use weak

2013-03-22 Thread Kai Tietz
Hi, this change is actual used by cygwin and is required for upcoming x64 cygwin target. ChangeLog 2013-03-22 Kai Tietz kti...@redhat.com * config/i386/cygming-crtbegin.c (__register_frame_info): Make weak. (__deregister_frame_info): Likewise. Tested for i686-pc-cygwin

[patch cygwin64]: Add and adjust some initial sources for x64 cygwin

2013-03-22 Thread Kai Tietz
Hi, Hi, the first part of required code-changes for upcoming cygwin x64 target. ChangeLog 2013-03-22 Kai Tietz kti...@redhat.com * config/i386/cygwin-stdint.h: Add support for cygwin x64 target. * config/i386/t-cygwin-w64: New file. * config/i386/cygwin-w64.h: New

[patch i386 windows]: Fix PR/52790 also required for workig upcoming cygwin x64 target

2013-03-22 Thread Kai Tietz
code-model default to medium also for native-windows x64 target. ChangeLog 2013-03-22 Kai Tietz kti...@redhat.com PR target/52790 * config/i386/cygming.h (SUB_TARGET_RECORD_STUB): New sub-target macro. * config/i386/i386-protos.h (i386_pe_record_stub): Add new prototype

Re: [patch libgcc]: Adjust cygming-crtbegin code to use weak

2013-03-22 Thread Kai Tietz
2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: On 22/03/2013 08:44, Kai Tietz wrote: Hi, this change is actual used by cygwin and is required for upcoming x64 cygwin target. ChangeLog 2013-03-22 Kai Tietz kti...@redhat.com * config/i386/cygming-crtbegin.c (__register_frame_info

Re: [patch cygwin64]: Add and adjust some initial sources for x64 cygwin

2013-03-22 Thread Kai Tietz
2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: On 22/03/2013 09:00, Kai Tietz wrote: (LIBGCJ_SONAME): Make name minor-build-version dependent. Tested for i686-pc-cygwin, and x86_64-pc-cygwin. Dave, please especially a look to LIBGCJ_SONAME change. I think we should include

Re: [patch cygwin64]: Add and adjust some initial sources for x64 cygwin

2013-03-22 Thread Kai Tietz
2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: On 22/03/2013 09:00, Kai Tietz wrote: (CXX_WRAP_SPEC_LIST): Undefine before define. @@ -73,6 +82,7 @@ /* To implement C++ function replacement we always wrap the cxx malloc-like operators. See N2800 #17.6.4.6

Re: [patch cygwin64]: Add and adjust some initial sources for x64 cygwin

2013-03-22 Thread Kai Tietz
2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: On 23/03/2013 00:16, Kai Tietz wrote: welcome, too. It would be even better if we could rethink actual the need of loading java-library within libgcc's cygwin's/mingw's crtbegin at all. I am actual not that sure, if we need this at all

Re: [patch libgcc]: Adjust cygming-crtbegin code to use weak

2013-03-22 Thread Kai Tietz
2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: On 23/03/2013 00:08, Kai Tietz wrote: 2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: Also, can you explain the motivation for this change? I don't see how it's going to work right; from what I remember, we don't have weak definitions

Re: [patch]: Add configure and required host/target definitions for new cygwin x64 target

2013-03-22 Thread Kai Tietz
2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: On 22/03/2013 09:56, Kai Tietz wrote: Hi, this patch adds required configure changes for new cygwin x64 target. Index: gcc/configure.ac === --- gcc/configure.ac (Revision 196898

Re: [patch cygwin64]: Add and adjust some initial sources for x64 cygwin

2013-03-22 Thread Kai Tietz
2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: On 23/03/2013 00:24, Kai Tietz wrote: 2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: On 23/03/2013 00:16, Kai Tietz wrote: welcome, too. It would be even better if we could rethink actual the need of loading java-library within libgcc's

Re: [patch cygwin64]: Add and adjust some initial sources for x64 cygwin

2013-03-24 Thread Kai Tietz
2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: On 23/03/2013 00:57, Kai Tietz wrote: 2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: On 23/03/2013 00:24, Kai Tietz wrote: 2013/3/23 Dave Korn dave.korn.cyg...@gmail.com: On 23/03/2013 00:16, Kai Tietz wrote: welcome, too. It would be even

Re: [patch cygwin64]: Add and adjust some initial sources for x64 cygwin

2013-03-27 Thread Kai Tietz
So I committed the patch with two modifications at revision 197168 as * config/i386/cygwin-stdint.h: Add support for cygwin x64 target. * config/i386/t-cygwin-w64: New file. * config/i386/cygwin-w64.h: New file. * config/i386/cygwin.h (EXTRA_OS_CPP_BUILTINS):

Re: [patch i386 windows]: Fix PR/52790 also required for workig upcoming cygwin x64 target

2013-03-28 Thread Kai Tietz
Ping

Re: [patch i386 windows]: Fix PR/52790 also required for workig upcoming cygwin x64 target

2013-04-02 Thread Kai Tietz
Ping ^2 - Original Message - From: Kai Tietz ktiet...@googlemail.com To: GCC Patches gcc-patches@gcc.gnu.org Cc: Richard Henderson r...@redhat.com Sent: Thursday, March 28, 2013 2:35:15 PM Subject: Re: [patch i386 windows]: Fix PR/52790 also required for workig upcoming cygwin x64

[patch tree-ssa-structalias.c]: Small finding in find_func_aliases function

2013-04-05 Thread Kai Tietz
Hello, while debugging I made the finding that in find_func_aliases rhsop might be used as NULL for gimple_assign_single_p items. It should be using for the gimple_assign_single_p instead directly the rhs1-item as argument to pass to get_constraint_for_rhs function. ChangeLog 2013-04-05 Kai

Re: [patch tree-ssa-structalias.c]: Small finding in find_func_aliases function

2013-04-08 Thread Kai Tietz
I haven't even applied it. Kai

[patch cygwin]: Replace use of TARGET_CYGWIN64 by TARGET_64BIT

2013-04-08 Thread Kai Tietz
Hi, this patch fixes an obvious typo in recently applied patch. ChangeLog 2013-04-08 Kai Tietz kti...@redhat.com * config/i386/cygwin.h (EXTRA_OS_CPP_BUILTINS): Replaced TARGET_CYGWIN64 by TARGET_64BIT. Applied to trunk as obvious fix. as revision 197593. Regards, Kai Index

Re: [patch libgcc]: Adjust cygming-crtbegin code to use weak

2013-04-09 Thread Kai Tietz
2013/4/9 Dave Korn dave.korn.cyg...@gmail.com: On 22/03/2013 08:44, Kai Tietz wrote: 2013-03-22 Kai Tietz kti...@redhat.com * config/i386/cygming-crtbegin.c (__register_frame_info): Make weak. (__deregister_frame_info): Likewise. Hi Kai, I read your explanation

[patch cygwin]: Add missing 64-bit cygwin triplet to config.gcc

2013-04-18 Thread Kai Tietz
Hi, an ChangeLog 2013-04-18 Kai Tietz kti...@redhat.com * config.gcc: Add x86_64-*-cygwin triplet. Tested for x86_64-pc-cygwin. I will apply it tomorrow, if there are no objections. Kai Index: config.gcc

[patch Ada]: Fix PR/55445 also for Ada's SEH-exception-mechanism

2013-04-23 Thread Kai Tietz
the additional check that SjLj is deactive. ChangeLog 2013-04-23 Kai Tietz kti...@redhat.com PR target/55445 * raise-gcc.c (__SEH__): Additional check that SjLj isn't active. Tested for x86_64-pc-cygwin, and x86_64-w64-mingw32. Ok for apply? Regards, Kai Index: raise-gcc.c

Re: [patch Ada]: Fix PR/55445 also for Ada's SEH-exception-mechanism

2013-04-23 Thread Kai Tietz
Ok, applied to trunk and 4.8 branch with your suggested change. Kai

{patch cp]: Disable hidden-inlines for pe-coff targets

2014-08-04 Thread Kai Tietz
Hi, This patch makes sure that visibility-option for hidden-inlines doesn't show any effect for pe-coff as for those targets no concept of hidden-visiblitly isn't present. ChangeLog 2014-08-04 Kai Tietz kti...@redhat.com * decl2.c (determine_hidden_inline): Always fail for PE-COFF

[patch cp]: Check for keep-inline-dllexport that we are operating on true inlines

2014-08-04 Thread Kai Tietz
Hi, This patch makes sure that we check for flag keep-inline-dllexport that we operate on true inlines as we do it for keep-inline-functions. ChangeLog 2014-08-04 Kai Tietz kti...@redhat.com * semantics.c (expand_or_defer_fn_1): Check for keep-inline-dllexport that we operate

Re: [PATCH] fix FTBFS with --target=i686-pc-cygwin

2014-08-12 Thread Kai Tietz
2014-08-12 20:34 GMT+02:00 Yaakov Selkowitz yselk...@redhat.com: There is a syntax error in r213009 causing a FTBFS in trunk with --target=i686-pc-cygwin. Patch attached, with which trunk now builds for said target. -- Yaakov Selkowitz Associate Software Engineer, ARM Red Hat, Inc. Patch

Re: [Patch] Enable libatomic for cygwin targets

2014-08-12 Thread Kai Tietz
2014-08-12 20:40 GMT+02:00 Yaakov Selkowitz yselk...@redhat.com: The attached patch enabled libatomic for Cygwin targets. check-target-libatomic for i686-pc-cygwin: Running gcc/libatomic/testsuite/libatomic.c/c.exp ... === libatomic Summary === # of expected passes

Re: [PATCH] Fix cygwin-stdint.h

2014-08-12 Thread Kai Tietz
2014-08-12 22:00 GMT+02:00 Yaakov Selkowitz yselk...@redhat.com: This patch fixes an error in r197168 where type definitions were made dependent on the host architecture instead of the target. This causes cross-compilation issues, particularly in a --host=x86_64-* --target=i686-pc-cygwin

Re: [PATCH] cygwin: fix --tsaware, add --large-address-aware

2014-08-13 Thread Kai Tietz
2014-08-13 7:14 GMT+02:00 Yaakov Selkowitz yselk...@redhat.com: This patch fixes two issues with LINK_SPEC on Cygwin targets: 1) --tsaware makes sense only for EXEs, not DLLs. 2) Under WoW64 (32-bit Cygwin on 64-bit Windows), Cygwin applications can access up to 4 GiB of address space (for

Re: [Patch] Fix crtstuff.c when compiling with mingw tools.

2014-08-13 Thread Kai Tietz
2014-08-13 22:51 GMT+02:00 Steve Ellcey sell...@mips.com: On Wed, 2014-08-13 at 20:42 +, Joseph S. Myers wrote: On Wed, 13 Aug 2014, Steve Ellcey wrote: This is a ping on a patch I sent out a while ago to fix the GCC build when building with the mingw toolset. This is not a review.

Re: [PATCH] fix FTBFS with --target=i686-pc-cygwin

2014-08-19 Thread Kai Tietz
Applied for you at rev. 214153. Kai 2014-08-12 20:37 GMT+02:00 Kai Tietz ktiet...@googlemail.com: 2014-08-12 20:34 GMT+02:00 Yaakov Selkowitz yselk...@redhat.com: There is a syntax error in r213009 causing a FTBFS in trunk with --target=i686-pc-cygwin. Patch attached, with which trunk now

Re: [Patch] Enable libatomic for cygwin targets

2014-08-19 Thread Kai Tietz
Applied for you at revision 214154. Kai

Re: [PATCH] Fix cygwin-stdint.h

2014-08-19 Thread Kai Tietz
Applied for you at revision 214156. Kai

Re: [PATCH] cygwin: fix --tsaware, add --large-address-aware

2014-08-19 Thread Kai Tietz
Applied for you at revision 214158. Kai

Re: [PATCH] cygwin: __cxa_atexit support

2014-08-19 Thread Kai Tietz
Patch is ok. Applied this patch at revision 214162 together with your followup-patch at revision 214161. Thanks, Kai

Re: [PATCH] cygwin: accept -pthread

2014-08-19 Thread Kai Tietz
Patch is ok. Applied this patch at revision 214161 together with your followup-patch at revision 214162. Thanks, Kai

Re: [PATCH] _cxa_thread_atexit fixes for Cygwin/MinGW-w64

2014-08-19 Thread Kai Tietz
Applied at revision 214163. Thanks, Kai

[patch g++.dg]: Disable some weak-tests for mingw-targets

2014-11-28 Thread Kai Tietz
Hi, this patch skips some test, which are trying to test non-existing weak-variant for mingw-targets. ChangeLog 2014-11-28 Kai Tietz kti...@redhat.com * g++.dg/abi/anon2.C: Skip for mingw targets. * g++.dg/abi/anon3.C: Likewise. * g++.dg/abi/thunk5.C: Likewise. * g++.dg/abi

Re: [patch g++.dg]: Disable some weak-tests for mingw-targets

2014-11-28 Thread Kai Tietz
2014-11-28 12:21 GMT+01:00 Rainer Orth r...@cebitec.uni-bielefeld.de: 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

Re: [patch testsuite gcc.dg]: Turn of ms-extensions for mingw target

2014-11-28 Thread Kai Tietz
2014-11-28 12:14 GMT+01:00 Rainer Orth r...@cebitec.uni-bielefeld.de: Hi Kai, 2014-11-28 Kai Tietz kti...@redhat.com * 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

Re: [patch testsuite g++]: Fixes for mingw-targets

2014-11-28 Thread Kai Tietz
2014-11-28 12:17 GMT+01:00 Rainer Orth r...@cebitec.uni-bielefeld.de: Hi Kai, 2014-11-28 Kai Tietz kti...@redhat.com * 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

Re: [patch testsuite gcc.dg]: Turn of ms-extensions for mingw target

2014-11-28 Thread Kai Tietz
2014-11-28 18:53 GMT+01:00 Joseph Myers jos...@codesourcery.com: 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=some ISO standard -pedantic (or -pedantic

Re: [patch testsuite gcc.dg]: Turn of ms-extensions for mingw target

2014-11-28 Thread Kai Tietz
2014-11-28 19:10 GMT+01:00 Joseph Myers jos...@codesourcery.com: 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

[patch c++]: Fix PR/64100 'A static assert using the the current class in a noexcept test leads to a segfault'

2014-12-04 Thread Kai Tietz
Hi, The issue is that lookup_destructor calls adjust_result_of_qualified_name_lookup with an NULL_TREE decl (returned by lookup_member). So error-message is missing. As already discussed in bug-tracker: ChangeLog 2014-12-04 Kai Tietz kti...@redhat.com PR c++/64100 * typeck.c

[patch c++]: Fix PR/64127 ICE on invalid: tree check: exprected identifier_node, have template_id_expr in cp_parser_diagnose_invalid_type_name

2014-12-04 Thread Kai Tietz
Hi, this patch fixes an ICE happening on invalid code for c++11. It is reasoned by accessing blindly identifier without checking that it is a declaration. ChangeLog 2014-12-04 Kai Tietz kti...@redhat.com PR c++/64127 * parser.c (cp_parser_diagnose_invalid_type_name): Check id

[patch c++]: Fix PR c++/64106 ICE on valid code

2014-12-04 Thread Kai Tietz
Hi, this patch adds INDIRECT_REF support to cxx_eval_store_expression handling. There is a different variant suggested by Marek, which adds additional operand-0 to ref, which looks to me wrong. ChangeLog gcc/cp 2014-12-04 Kai Tietz kti...@redhat.com PR c++/64106 * constexpr.c

Re: [patch c++]: Fix PR/64100 'A static assert using the the current class in a noexcept test leads to a segfault'

2014-12-04 Thread Kai Tietz
2014-12-04 16:46 GMT+01:00 Marek Polacek pola...@redhat.com: On Thu, Dec 04, 2014 at 04:00:34PM +0100, Kai Tietz wrote: Hi, The issue is that lookup_destructor calls adjust_result_of_qualified_name_lookup with an NULL_TREE decl (returned by lookup_member). So error-message is missing

Re: [patch c++]: Fix PR/64127 ICE on invalid: tree check: exprected identifier_node, have template_id_expr in cp_parser_diagnose_invalid_type_name

2014-12-04 Thread Kai Tietz
2014-12-04 16:47 GMT+01:00 Marek Polacek pola...@redhat.com: On Thu, Dec 04, 2014 at 04:12:02PM +0100, Kai Tietz wrote: Hi, this patch fixes an ICE happening on invalid code for c++11. It is reasoned by accessing blindly identifier without checking that it is a declaration. ChangeLog

Re: [patch c++]: Fix PR/64127 ICE on invalid: tree check: exprected identifier_node, have template_id_expr in cp_parser_diagnose_invalid_type_name

2014-12-04 Thread Kai Tietz
So added testcase for this pr (its c++98 only) So: ChangeLog testsuite 2014-12-04 Kai Tietz kti...@redhat.com PR c++/64127 * g++.dg/cpp/pr64127.C: New file. Tested on x86_64-unknown-linux-gnu. Ok to apply prior posted patch plus this new testcase? Regards, Kai Index: gcc/gcc

Re: [patch c++]: Fix PR/64127 ICE on invalid: tree check: exprected identifier_node, have template_id_expr in cp_parser_diagnose_invalid_type_name

2014-12-04 Thread Kai Tietz
2014-12-04 20:35 GMT+01:00 Jason Merrill ja...@redhat.com: On 12/04/2014 10:12 AM, Kai Tietz wrote: else if (cxx_dialect cxx11 +DECL_P (id) !strcmp (IDENTIFIER_POINTER (id), thread_local)) This doesn't make any sense: If it's a decl it isn't an identifier

Re: [patch c++]: Fix PR/64127 ICE on invalid: tree check: exprected identifier_node, have template_id_expr in cp_parser_diagnose_invalid_type_name

2014-12-04 Thread Kai Tietz
Updated version checking the right thing ... ChangeLog gcc/cp 2014-12-04 Kai Tietz kti...@redhat.com PR c++/64127 * parser.c (cp_parser_diagnose_invalid_type_name): Check id for being an identifier before accessing it. Tested on x86_64-unknown-linux-gnu. Ok for apply together

Re: [patch c++]: Fix PR/64100 'A static assert using the the current class in a noexcept test leads to a segfault'

2014-12-04 Thread Kai Tietz
Ok, Additional change for testcase mentioned in PR. ChangeLog 2014-12-04 Kai Tietz kti...@redhat.com PR c++/641000 * g++.dg/template/pr64100.C: New file. Ok for apply? Regards, Kai Index: gcc/gcc/testsuite/g++.dg/template/pr64100.C

[patch c++]: Fix PR/63996

2014-12-12 Thread Kai Tietz
Hi, The loop-expression loops endless in c++14's case for cases the statement-list isn't constant. Bug 63996 - Infinite loop in invalid C++14 constexpr fn ChangeLog 2014-12-12 Kai Tietz kti...@redhat.com PR c++/63996 * constexpr.c (cxx_eval_loop_expr): Don't loop endless on none

[patch c++]: Fix 61228 - noexcept(expression) causes internal compiler error

2014-12-12 Thread Kai Tietz
Hi, following patch fixes reported issue. Tested for x86_64-w64-mingw32. Ok for apply? Regards, Kai ChangeLog 2014-12-12 Kai Tietz kti...@redhat.com PR c++/61228 * call.c (set_flags_from_callee): Assume no throw by deferred noexcept. 2014-12-12 Kai Tietz kti...@redhat.com

Re: [patch c++]: Fix 61228 - noexcept(expression) causes internal compiler error

2014-12-12 Thread Kai Tietz
2014-12-12 21:15 GMT+01:00 Jason Merrill ja...@redhat.com: I think it would be better to call maybe_instantiate_noexcept so that we can have a definite answer. Jason Hmm, for case that decl != NULL_TREE, this is ok. But what if decl is NULL_TREE? Kai

Re: [patch c++]: Fix PR/63996

2014-12-15 Thread Kai Tietz
2014-12-15 11:48 GMT+01:00 Paolo Carlini paolo.carl...@oracle.com: ... committed as obvious the below. Paolo. / Thanks Kai

[patch c++]: Fix PR/61198: Crash when selecting specializations through aliases.

2014-12-18 Thread Kai Tietz
Hi, this patch adds handling of aliases within templates and tries to resolve specialization for them. ChangeLog 2014-12-18 Kai Tietz kti...@redhat.com PR c++/61198 * pt.c (retrieve_specialization): Handle using. Tested on x86_64-w64-mingw32. Ok for apply? Regards, Kai ChangeLog

Re: [patch c++]: Fix PR/61198: Crash when selecting specializations through aliases.

2014-12-18 Thread Kai Tietz
2014-12-18 18:26 GMT+01:00 Jason Merrill ja...@redhat.com: On 12/18/2014 10:10 AM, Kai Tietz wrote: + if (TMPL_ARGS_DEPTH (args) TMPL_PARMS_DEPTH (tmpl_parms)) +args = get_innermost_template_args +(args, TMPL_PARMS_DEPTH (tmpl_parms)); It seems unlikely to be correct

Re: [patch c++]: Fix PR/61198: Crash when selecting specializations through aliases.

2014-12-19 Thread Kai Tietz
2014-12-19 0:14 GMT+01:00 Jason Merrill ja...@redhat.com: On 12/18/2014 01:16 PM, Kai Tietz wrote: Well, in general I would have assumed to be able to get alias decl of tmpl. Wasn't able to find a simple way to get it. So, by looking into source I found that most cases handling args tmpl

Re: [patch c++]: Fix PR/61198: Crash when selecting specializations through aliases.

2014-12-19 Thread Kai Tietz
2014-12-19 17:46 GMT+01:00 H.J. Lu hjl.to...@gmail.com: Did you forget to check in the testcase? No, see rev 218956. -- H.J.

Re: [patch c++]: Fix PR/61198: Crash when selecting specializations through aliases.

2014-12-19 Thread Kai Tietz
2014-12-19 18:22 GMT+01:00 Paolo Carlini paolo.carl...@oracle.com: Hi, On 12/19/2014 05:48 PM, Kai Tietz wrote: 2014-12-19 17:46 GMT+01:00 H.J. Lu hjl.to...@gmail.com: Did you forget to check in the testcase? No, see rev 218956. But you want to move it to the cpp0x directory

Re: Fix a MinGW warning in libiberty/strerror.c

2015-01-16 Thread Kai Tietz
Hi Eli, patch is reasonable and ok for me. Thanks Kai 2015-01-16 12:18 GMT+01:00 Eli Zaretskii e...@gnu.org: Ping! Date: Fri, 02 Jan 2015 12:54:47 +0200 From: Eli Zaretskii e...@gnu.org When compiling GDB 7.8.1, I get this warning in libiberty: gcc -c -DHAVE_CONFIG_H -O0 -g3

Re: Fix a MinGW warning in libiberty/strerror.c

2015-01-16 Thread Kai Tietz
2015-01-16 12:50 GMT+01:00 Eli Zaretskii e...@gnu.org: Date: Fri, 16 Jan 2015 12:34:25 +0100 From: Kai Tietz ktiet...@googlemail.com Cc: GCC Patches gcc-patches@gcc.gnu.org, gdb-patc...@sourceware.org gdb-patc...@sourceware.org Hi Eli, patch is reasonable and ok for me. Thanks. Do

[patch c++]: Fix for PR/65390

2015-03-16 Thread Kai Tietz
Hi, this patch avoids the attempt to create user-aligned-type for variants original and main-variants type-alignment differ and original type isn't user-aligned. Not sure if this is the preferred variant, we could create for such cases an aligned-type without setting user-align. But as this

Re: [patch c++]: Fix for PR/65390

2015-03-17 Thread Kai Tietz
2015-03-17 13:36 GMT+01:00 Jason Merrill ja...@redhat.com: On 03/16/2015 03:22 PM, Kai Tietz wrote: 2015-03-16 19:07 GMT+01:00 Jason Merrill ja...@redhat.com: If there is an alignment mismatch without user intervention, there is a problem, we can't just ignore it. Where we run into trouble

Re: [patch c++]: Fix for PR/65390

2015-03-20 Thread Kai Tietz
Hello, the problem here is that for cases of vla-array-types, the types don't get finally layouted in build_cplus_array_type. So the type-alignment isn't set in such cases for the resulting type. ChangeLog 2015-03-20 Kai Tietz kti...@redhat.com PR c++/65390 * tree.c (strip_typedefs

Re: [patch c++]: Fix for PR/65390

2015-03-16 Thread Kai Tietz
2015-03-16 19:07 GMT+01:00 Jason Merrill ja...@redhat.com: If there is an alignment mismatch without user intervention, there is a problem, we can't just ignore it. Where we run into trouble is with array types where the version built earlier has not been laid out yet but the new one has

[patch libgomp]: Fix PR 64972

2015-03-25 Thread Kai Tietz
Hi, ChangeLog 2015-03-25 Kai Tietz kti...@redhat.com PR libgomp/64972 * oacc-parallel.c (GOACC_parallel): Use PRIu64 if available. (GOACC_data_start): Likewise. * target.c (gomp_map_vars): Likewise. Tested for i686-w64-mingw32. Fix got preapproved by Jakub, so I will commit

[patch emutls]:Fix PR 65566

2015-03-31 Thread Kai Tietz
Hi, This patch avoids that we try to operate on function-decl's cfun equal to NULL within lower_emutls_function_body. ChangeLog 2015-03-31 Kai Tietz kti...@redhat.com PR target/65566 * tree-emutls.c (lower_emutls_function_body): Don't try to operate on node's decl function

Re: [patch emutls]:Fix PR 65566

2015-03-31 Thread Kai Tietz
2015-03-31 13:42 GMT+02:00 Jakub Jelinek ja...@redhat.com: On Tue, Mar 31, 2015 at 01:35:56PM +0200, Kai Tietz wrote: Hi, This patch avoids that we try to operate on function-decl's cfun equal to NULL within lower_emutls_function_body. If DECL_STRUCT_FUNCTION (node-decl) is already NULL

Re: [patch c++]: Fix for PR/65390

2015-03-31 Thread Kai Tietz
2015-03-31 14:34 GMT+02:00 Marek Polacek pola...@redhat.com: On Tue, Mar 31, 2015 at 02:32:32PM +0200, Marek Polacek wrote: On Tue, Mar 31, 2015 at 02:25:14PM +0200, Kai Tietz wrote: Hi, I had tried same approach as Marek. For me it solved the PR, but caused other regressions

Re: [patch c++]: Fix for PR/65390

2015-03-31 Thread Kai Tietz
Hi, I had tried same approach as Marek. For me it solved the PR, but caused other regressions on boostrap. So I dropped the way via dependent_type_p. Well, this bootstrap-issue might be caused by some local changes I had forgot to remove, but I doubt it. Marek, have you tried to do a boostrap

Re: [patch]: Fix Bug 65038 - [regression 5] Unable to find ftw.h for libgcov-util.c

2015-02-27 Thread Kai Tietz
2015-02-27 14:48 GMT+01:00 H.J. Lu hjl.to...@gmail.com: On Thu, Feb 26, 2015 at 6:49 AM, Kai Tietz ktiet...@googlemail.com wrote: Hi, This is the remaining fix for re-enabling native boostrap for Windows-variant of gcc without disabling -Werror for libgcc. ChangeLog 2015-02-26 Kai Tietz

Re: [patch]: Fix Bug 65038 - [regression 5] Unable to find ftw.h for libgcov-util.c

2015-02-27 Thread Kai Tietz
Applied at rev.revision 221055. -- Kai

Re: [patch]: [Bug tree-optimization/61917] [4.9/5 Regression] ICE on valid code at -O3 on x86_64-linux-gnu in vectorizable_reduction, at tree-vect-loop.c:4913

2015-02-25 Thread Kai Tietz
2015-02-25 12:35 GMT+01:00 Richard Biener richard.guent...@gmail.com: On Wed, Feb 25, 2015 at 12:05 PM, Kai Tietz ktiet...@googlemail.com wrote: 2015-02-25 11:57 GMT+01:00 Richard Biener richard.guent...@gmail.com: On Wed, Feb 25, 2015 at 11:06 AM, Kai Tietz ktiet...@googlemail.com wrote

Re: [patch]: [Bug tree-optimization/61917] [4.9/5 Regression] ICE on valid code at -O3 on x86_64-linux-gnu in vectorizable_reduction, at tree-vect-loop.c:4913

2015-02-25 Thread Kai Tietz
Hello, So, I did full regression-test for following patch: ChangeLog 2015-02-25 Richard Biener rguent...@suse.de Kai Tietz kti...@redhat.com PR tree-optimization/61917 * tree-vect-loop.c (vectorizable_reduction): Allow vect_internal_def without reduction to exit graceful

[patch]: Fix PR target/64212

2015-02-25 Thread Kai Tietz
for clones we want that they getting interposable. Therefore - as suggested by Honza - we need to make explicit sure that we set DECL_DLLIMPORT_P() explicit to 0 in symtab. ChangeLog 2015-02-25 Kai Tietz kti...@redhat.com PR target/64212 * symtab.c (symtab::make_decl_local): Set DECL_IMPORT_P

Re: [patch]: Fix PR target/64212

2015-02-25 Thread Kai Tietz
Applied at revision 22098 to trunk. Jan approved patch on IRC. Regards, Kai

[patch]: Fix regression caused by fix for 61917

2015-02-25 Thread Kai Tietz
Hi, The patch didn't handled the case for dt being vect_constant_def, where of course the reduc_def_stmt is NULL. By checking for NULL before testing for PHI, we now fallback for such cases to old behavior and return in the next if-statment. 2015-02-25 Richard Biener rguent...@suse.de Kai

[patch]: Fix Bug 65038 - [regression 5] Unable to find ftw.h for libgcov-util.c

2015-02-26 Thread Kai Tietz
Hi, This is the remaining fix for re-enabling native boostrap for Windows-variant of gcc without disabling -Werror for libgcc. ChangeLog 2015-02-26 Kai Tietz kti...@redhat.com PR target/65038 * config.in: Regenerated. * configure: Likewise. * configure.ac (AC_HEADER_STDC

Re: [patch c-family]: Fix Bug 35330 - [4.8/4.9/5 regression] ICE with invalid pragma weak

2015-02-26 Thread Kai Tietz
2015-02-26 19:53 GMT+01:00 Marek Polacek pola...@redhat.com: On Thu, Feb 26, 2015 at 07:28:02PM +0100, Kai Tietz wrote: Hi, This patch addresses the reported ICE about #pragma weak used on declarations not var or function. ChangeLog 2015-02-26 Kai Tietz kti...@redhat.com * c

[patch c-family]: Fix Bug 35330 - [4.8/4.9/5 regression] ICE with invalid pragma weak

2015-02-26 Thread Kai Tietz
Hi, This patch addresses the reported ICE about #pragma weak used on declarations not var or function. ChangeLog 2015-02-26 Kai Tietz kti...@redhat.com * c-pragma.c (handle_pragma_weak): Do not try to creat weak/alias of declarations not being function, or variable

Re: [C++ PATCH] Fix ICE in C++14 with null RETURN_EXPR (PR c++/65202)

2015-02-26 Thread Kai Tietz
Hi Marek, I have similiar change on the c++-delayed-fold branch. 2015-02-26 13:22 GMT+01:00 Marek Polacek pola...@redhat.com: We ICE on this invalid testcase in C++14 because in C++14 a function returning void is a valid constexpr function, so adl_swap is registered as one, while in C++11 it

[patch]: [Bug tree-optimization/61917] [4.9/5 Regression] ICE on valid code at -O3 on x86_64-linux-gnu in vectorizable_reduction, at tree-vect-loop.c:4913

2015-02-25 Thread Kai Tietz
Hello, ChangeLog 2015-02-25 Kai Tietz kti...@redhat.com PR tree-optimization/61917 * tree-vect-loop.c (vectorizable_reduction): Allow vect_internal_def without reduction to exit graceful. ChagneLog testsuite/ 2015-02-25 Kai Tietz kti...@redhat.com PR tree-optimization

Re: [patch]: [Bug tree-optimization/61917] [4.9/5 Regression] ICE on valid code at -O3 on x86_64-linux-gnu in vectorizable_reduction, at tree-vect-loop.c:4913

2015-02-25 Thread Kai Tietz
2015-02-25 11:57 GMT+01:00 Richard Biener richard.guent...@gmail.com: On Wed, Feb 25, 2015 at 11:06 AM, Kai Tietz ktiet...@googlemail.com wrote: Hello, ChangeLog 2015-02-25 Kai Tietz kti...@redhat.com PR tree-optimization/61917 * tree-vect-loop.c (vectorizable_reduction): Allow

Re: C++ delayed folding branch review

2015-04-24 Thread Kai Tietz
2015-04-24 6:22 GMT+02:00 Jason Merrill ja...@redhat.com: + expr = fold (expr); /* This may happen, because for LHS op= RHS we preevaluate RHS and create C_MAYBE_CONST_EXPR SAVE_EXPR RHS, which means we could no longer see the code of the EXPR. */ if (TREE_CODE (expr) ==

Re: C++ delayed folding branch review

2015-04-28 Thread Kai Tietz
2015-04-24 20:25 GMT+02:00 Jason Merrill ja...@redhat.com: On 04/24/2015 09:46 AM, Kai Tietz wrote: Sure, we can use here instead *_fully_fold, but for what costs? In general we need to deal here a simple one-level fold for simplifying constant-values, and/or removing useless type

Re: C++ delayed folding branch review

2015-06-12 Thread Kai Tietz
Hello Jason, Thanks for the review. I addressed a lot of your comments directly on svn-branch. See revision r224439. - Ursprüngliche Mail - Generally, it seems like most of my comments from April haven't been addressed yet. Yes, most of them. @@ -3023,13 +3023,14 @@

[patch c++]: Fix PR 59759

2015-05-26 Thread Kai Tietz
ChangeLog 2015-05-26 Kai Tietz kti...@redhat.com PR c++/59759 * pt.c (unify): Don't ICE on VAR_DECL. Regression tested on x86_64-w64-mingw32. Ok for apply? Regards, Kai Index: pt.c === --- pt.c(Revision 219014) +++ pt.c

Re: C++ delayed folding branch review

2015-08-24 Thread Kai Tietz
Hello Jason, after a longer delay the answer to your question. 2015-08-03 17:39 GMT+02:00 Jason Merrill ja...@redhat.com: On 08/03/2015 05:42 AM, Kai Tietz wrote: 2015-08-03 5:49 GMT+02:00 Jason Merrill ja...@redhat.com: On 07/31/2015 05:54 PM, Kai Tietz wrote: The STRIP_NOPS-requirement

<    2   3   4   5   6   7   8   >