[PATCH] doc clarification: DONE and FAIL in define_split and define_peephole2

2018-07-05 Thread Paul Koning
ribe those cases. Ok for trunk? paul ChangeLog: 2018-07-05 Paul Koning * doc/md.texi (define_split): Document DONE and FAIL. Describe interaction with usual "#" output template in define_insn_and_split. (define_peephole2): Document DONE a

Re: [PATCH] doc clarification: DONE and FAIL in define_split and define_peephole2

2018-07-06 Thread Paul Koning
uly available for the >> +particular inputs it was given, and the input insns are left unchanged. > > If it FAILs, GCC will try to apply later define_peehole2s instead. > (This is in contrast to define_split, so it's a bit inconsistent. > Would be easy to make define_split behav

Re: [PATCH] doc clarification: DONE and FAIL in define_split and define_peephole2

2018-07-06 Thread Paul Koning
> On Jul 6, 2018, at 12:20 PM, Richard Sandiford > wrote: > > Double empty line. > > OK otherwise, thanks. (Think this counts as a gen* patch.) > > Richard Thanks. Committed as shown below. paul ChangeLog: 2018-07-06 Paul Koning * d

[PATCH, doc] Small clarification on define_subst

2018-07-08 Thread Paul Koning
In doing CCmode work I was confused how define_subst handles cases where the same argument appears more than once. The attached clarifies this. Ok for trunk? paul ChangeLog: 2018-07-08 Paul Koning * doc/md.texi (define_subst): Document how multiple occurrences of

[PATCH, committed] Improve code generation for pdp11 target

2018-07-09 Thread Paul Koning
This patch improves the generated code for the pdp11 target. Committed. paul ChangeLog: 2018-07-09 Paul Koning * config/pdp11/pdp11.c (pdp11_addr_cost): New function. (pdp11_insn_cost): New function. (pdp11_md_asm_adjust): New function

[PATCH] doc: add missing "mode" type attribute

2018-07-10 Thread Paul Koning
butes. Ok for trunk? paul ChangeLog: 2018-07-10 Paul Koning * doc/extend.texi (Common Variable Attributes): Move "mode" into alphabetical order. (Common Type Attributes): Add "mode&

[PATCH] doc: update looping constructs

2018-07-11 Thread Paul Koning
This patch removes the obsolete documentation for decrement_and_branch_until_zero. It also adds detail to the description for doloop_end. In particular, it describes the required form of the conditional branch part of the pattern. Ok for trunk? paul ChangeLog: 2018-07-11 Paul

Re: [PATCH] doc: update looping constructs

2018-07-12 Thread Paul Koning
> On Jul 12, 2018, at 12:55 PM, Jeff Law wrote: > > On 07/11/2018 06:20 PM, Paul Koning wrote: >> This patch removes the obsolete documentation for >> decrement_and_branch_until_zero. It also adds detail to the description for >> doloop_end. In particular, it

Re: [PATCH] doc: update looping constructs

2018-07-12 Thread Paul Koning
> On Jul 12, 2018, at 1:42 PM, Jeff Law wrote: > > On 07/12/2018 11:17 AM, Paul Koning wrote: >> >> >>> On Jul 12, 2018, at 12:55 PM, Jeff Law wrote: >>> >>> On 07/11/2018 06:20 PM, Paul Koning wrote: >&g

[PATCH, committed] Fix typo in pdp11 target

2018-07-12 Thread Paul Koning
This fixes a typo in the output of a ".set" directive. Committed. paul ChangeLog: 2018-07-12 Paul Koning * config/pdp11/pdp11.c (pdp11_output_def): Fix typo in .set directive. Index: config/pdp

[PATCH, committed] Fix rtl check error in pdp11

2018-07-14 Thread Paul Koning
When building with checking enabled, there were check failures in pdp11_rtx_costs. This patch fixes two errors. Committed. paul ChangeLog: 2018-07-14 Paul Koning * config/pdp11/pdp11.c (pdp11_rtx_costs): Bugfixes. Index: config/pdp11/pdp11.c

Re: [2/5] C-SKY port: Backend implementation

2018-07-25 Thread Paul Koning
> On Jul 25, 2018, at 12:50 AM, Jeff Law wrote: > ... >>> It did. See TARGET_CUSTOM_FUNCTION_DESCRIPTORS and the (relatively few) >>> ports that define it. >> >> Hmmm, I completely failed to make that connection from the docs -- the >> whole description of that hook is pretty gibberishy

Re: [PATCH 1/7] Add __builtin_speculation_safe_value

2018-07-26 Thread Paul Koning
> On Jul 26, 2018, at 7:34 PM, Joseph Myers wrote: > > On Wed, 25 Jul 2018, Richard Earnshaw (lists) wrote: > Port maintainers DO need to decide what to do about speculation, even if it is explicitly that no mitigation is needed. >>> >>> Agreed. But I didn't yet see a request for

Re: [PATCH 09/11] pdp11 - example of a port not needing a speculation barrier

2018-07-27 Thread Paul Koning
> On Jul 27, 2018, at 5:37 AM, Richard Earnshaw > wrote: > > > This patch is intended as an example of all that is needed if the > target system doesn't support CPUs that have speculative execution. > I've chosen the pdp11 port on the basis that it's old enough that this > is likely to be tr

Re: [patch] improve internals documentation for nested function descriptors

2018-07-27 Thread Paul Koning
> On Jul 27, 2018, at 4:39 PM, Sandra Loosemore wrote: > > Apropos of the discussion about improving the docs for > TARGET_CUSTOM_FUNCTION_DESCRIPTORS in the context of the C-SKY port > submission, > > https://gcc.gnu.org/ml/gcc-patches/2018-07/msg01454.html > > here is the patch I've come

Re: Deprecating cc0 (and consequently cc0 targets)

2019-10-30 Thread Paul Koning
> On Oct 30, 2019, at 2:24 PM, Jeff Law wrote: > > On 10/30/19 2:12 AM, Richard Biener wrote: >> On Tue, Oct 29, 2019 at 8:34 PM Jeff Law wrote: > >> >> I think the wiki has better examples. That said, I wonder how much can >> be automated here, for example when just considering CCmode (m6

Re: [RFC] Characters per line: from punch card (80) to line printer (132) (was: [Patch][OpenMP/OpenACC/Fortran] Fix mapping of optional (present|absent) arguments)

2019-12-05 Thread Paul Koning
> On Dec 5, 2019, at 11:17 AM, Joseph Myers wrote: > > On Thu, 5 Dec 2019, Thomas Schwinge wrote: > >> In the relevant session at the GNU Tools Cauldron 2019, Michael Meissner >> stated that even he is not using a 80 x 24 terminal anymore, and that >> should tell us something. ;-) >> >> So,

Re: [PATCH] Turn on LRA on all targets

2024-02-15 Thread Paul Koning
> On May 15, 2023, at 5:09 PM, Maciej W. Rozycki wrote: > > ... > > I may choose to implement a non-DWARF unwinder instead, as the VAX stack > frame is always fully described by the hardware and there is never ever a > need for debug information to be able to decode any VAX stack frame (the

Re: [PATCH] Turn on LRA on all targets

2024-02-15 Thread Paul Koning
> On Feb 15, 2024, at 5:56 PM, Segher Boessenkool > wrote: > > On Thu, Feb 15, 2024 at 07:34:32PM +, Sam James wrote: >> I have now started doing this in PR113932. > > Thank you! > > Segher Presumably this isn't for version 14 since it's in a late stage, right? I have my bits about r

Re: [PATCH] Turn on LRA on all targets

2024-02-16 Thread Paul Koning
> On Feb 16, 2024, at 6:34 AM, Maciej W. Rozycki wrote: > > On Thu, 15 Feb 2024, Paul Koning wrote: > >>> On May 15, 2023, at 5:09 PM, Maciej W. Rozycki wrote: >>> >>> ... >>> >>> I may choose to implement a non-DWARF unwinder inste

Re: [PATCH] Fix PR ada/111909 On Darwin, determine filesystem case sensitivity at runtime

2023-11-22 Thread Paul Koning
> On Nov 22, 2023, at 8:54 AM, Simon Wright wrote: > > On 21 Nov 2023, at 23:13, Iain Sandoe wrote: > >>> #if defined (__APPLE__) >>> -#include >> >> If removing unistd.h is intentional (i.e. you determined that it’s no longer >> needed for Darwin), then we should make that a separate patc

Re: [committed] PATCH for Re: Stepping down as maintainer for ARC and Epiphany

2024-05-21 Thread Paul Koning
> On May 21, 2024, at 9:57 AM, Jeff Law wrote: > > > > On 5/21/24 12:05 AM, Richard Biener via Gcc wrote: >> On Mon, May 20, 2024 at 4:45 PM Gerald Pfeifer wrote: >>> >>> On Wed, 5 Jul 2023, Joern Rennecke wrote: I haven't worked with these targets in years and can't really do se

Re: LRA: Fix setup_sp_offset

2024-08-26 Thread Paul Koning
> On Aug 26, 2024, at 10:14 AM, Michael Matz wrote: > > Hello, > > On Sun, 25 Aug 2024, Jeff Law wrote: > >>> 550: [--sp] = 0 sp_off = 0 {pushexthisi_const} >>> 551: [--sp] = 37sp_off = -4 {pushexthisi_const} >>> 552: [--sp] = r37 sp_off = -8 {movsi_m

Re: LRA: Fix setup_sp_offset

2024-08-26 Thread Paul Koning
> On Aug 26, 2024, at 10:40 AM, Michael Matz wrote: > > Hello, > > On Mon, 26 Aug 2024, Paul Koning wrote: > >>>>> 550: [--sp] = 0 sp_off = 0 {pushexthisi_const} >>>>> 551: [--sp] = 37sp_off = -4 {pushexthisi_const

Re: PING^1 [PATCH 30/52] pdp11: Remove macro LONG_DOUBLE_TYPE_SIZE

2024-06-13 Thread Paul Koning
What is the effect of this change? The original code intended to have "float" mean a 32 bit value, and "double" a 64 bit value. There aren't any larger floats, so I defined the long double size as 64 also. Is the right answer not to define it? That part I understand, but why does the patch a

Re: [PATCH 30/52 v2] pdp11: Remove macro {FLOAT,DOUBLE,LONG_DOUBLE}_TYPE_SIZE

2024-06-14 Thread Paul Koning
Ok, I understand better now. But if those macros are supposed to be replaced by hook functions, could you make that replacement part of the proposed patch? paul > On Jun 13, 2024, at 11:22 PM, Kewen.Lin wrote: > > Hi Paul, > > on 2024/6/14 04:07, Paul Koning wrote:

Re: [PATCH 30/52 v2] pdp11: Remove macro {FLOAT,DOUBLE,LONG_DOUBLE}_TYPE_SIZE

2024-06-17 Thread Paul Koning
Thanks Kewen. Given that background, the patch is OK. paul > On Jun 16, 2024, at 10:01 PM, Kewen.Lin wrote: > > Hi Paul, > > on 2024/6/14 23:20, Paul Koning wrote: >> Ok, I understand better now. But if those macros are supposed to be >> replaced by hoo

Re: [PATCH] Hard register asm constraint

2024-06-25 Thread Paul Koning
> On Jun 24, 2024, at 1:50 AM, Stefan Schulze Frielinghaus > wrote: > > Ping. > > On Mon, Jun 10, 2024 at 07:19:19AM +0200, Stefan Schulze Frielinghaus wrote: >> Ping. >> >> On Fri, May 24, 2024 at 11:13:12AM +0200, Stefan Schulze Frielinghaus wrote: >>> This implements hard register constr

Re: [PATCH] Hard register asm constraint

2024-06-25 Thread Paul Koning
> On Jun 25, 2024, at 12:04 PM, Stefan Schulze Frielinghaus > wrote: > > On Tue, Jun 25, 2024 at 10:03:34AM -0400, Paul Koning wrote: >> >>>>> ... >>>>> could be rewritten into >>>>> >>>>> int test (int

Re: [PATCH] Hard register asm constraint

2024-06-26 Thread Paul Koning
> On Jun 26, 2024, at 8:54 AM, Stefan Schulze Frielinghaus > wrote: > > On Tue, Jun 25, 2024 at 01:02:39PM -0400, Paul Koning wrote: >> >> >>> On Jun 25, 2024, at 12:04 PM, Stefan Schulze Frielinghaus >>> wrote: >>> >>>

Re: [PATCH 1/3][v2] Add TARGET_MODE_CAN_TRANSFER_BITS

2024-07-30 Thread Paul Koning
> On Jul 30, 2024, at 6:17 AM, Richard Biener wrote: > > The following adds a target hook to specify whether regs of MODE can be > used to transfer bits. The hook is supposed to be used for value-numbering > to decide whether a value loaded in such mode can be punned to another > mode instead

Re: Problem exposed by recent ARM multiply changes

2019-09-26 Thread Paul Koning
> On Sep 26, 2019, at 12:01 PM, Jeff Law wrote: > > On 9/26/19 9:47 AM, Jakub Jelinek wrote: >> On Thu, Sep 26, 2019 at 09:39:31AM -0600, Jeff Law wrote: >>> Right. My point is that the multiplication patterns are an exception as >>> well. >> >> Do you have some evidence for that? > It's in

Re: [PATCH] regrename: Use PC instead of CC0 to hide operands

2019-10-01 Thread Paul Koning
On Oct 1, 2019, at 5:14 AM, Segher Boessenkool wrote: > > The regrename pass temporarily changes some operand RTL to CC0 so that > note_stores and scan_rtx don't see those operands. CC0 is deprecated > and we want to remove it, so we need to use something else here. > PC fits the bill fine. CC

Re: Fix ALL_REGS thinko in initialisation of function_used_regs

2019-10-03 Thread Paul Koning
> On Oct 3, 2019, at 9:12 AM, Richard Earnshaw (lists) > wrote: > > On 03/10/2019 10:48, Segher Boessenkool wrote: >>> ... >> But ALL_REGS should contain *all* registers. The union of any two register >> classes has to be contained in some register class, so every register class >> has to be

Re: [PATCH 3/4] Set costs for jumps in combine

2019-11-21 Thread Paul Koning
> On Nov 21, 2019, at 7:42 PM, Segher Boessenkool > wrote: > > ... > Maybe i386 should implement the insn_cost hook as well? For most targets > that is a lot simpler to get right than rtx_cost, but allowing memory in > many insns and all the different insn lengths complicates matters. At >

Re: [PATCH] Use getentropy() for seeding PRNG

2018-08-03 Thread Paul Koning
> On Aug 3, 2018, at 9:19 AM, Janne Blomqvist wrote: > > The getentropy function, found on Linux, OpenBSD, and recently also > FreeBSD, can be used to get random bytes to initialize the PRNG. It > is similar to the traditional way of reading from /dev/urandom, but > being a system call rather

Re: [PATCH] Add sinh(tanh(x)) and cosh(tanh(x)) rules

2018-08-07 Thread Paul Koning
> On Aug 7, 2018, at 4:00 PM, Giuliano Augusto Faulin Belinassi > wrote: > > Related with bug 86829, but for hyperbolic trigonometric functions. > This patch adds substitution rules to both sinh(tanh(x)) -> x / sqrt(1 > - x*x) and cosh(tanh(x)) -> 1 / sqrt(1 - x*x). Notice that the both > for

Re: [PATCH] Add sinh(tanh(x)) and cosh(tanh(x)) rules

2018-08-08 Thread Paul Koning
4:42 PM, Giuliano Augusto Faulin Belinassi > wrote: > > That is a good question because I didn't know that such targets > exists. Any suggestion? > > > On Tue, Aug 7, 2018 at 5:29 PM, Paul Koning wrote: >> >> >>> On Aug 7, 2018, at 4:00 PM, Giu

Re: [PATCH] Add sinh(tanh(x)) and cosh(tanh(x)) rules

2018-08-08 Thread Paul Koning
tch code and comments are still correct. > > On Wed, Aug 8, 2018 at 10:58 AM, Paul Koning wrote: >> Now I'm puzzled. >> >> I don't see how an infinite would show up in the original expression. I >> don't know hyperbolic functions, so I just constructed

Re: [PATCH 3/3] or1k: gcc: initial support for openrisc

2018-08-31 Thread Paul Koning
> On Aug 30, 2018, at 9:02 PM, Jeff Law wrote: > > On 08/30/2018 10:58 AM, Richard Henderson wrote: >> On 08/28/2018 07:13 AM, Jeff Law wrote: >>> Please consider using function descriptors rather than trampolines. >>> This allows you to make the stack non-executable at all times which is >>>

[PATCH, committed] fix a number of pdp11 target test failures

2018-05-25 Thread Paul Koning
This fixes a number of test failures due to test cases that are too large for pdp11 and should be skipped. Also one test case that asks for alignment larger than what pdp11 supports. paul 2018-05-25 Paul Koning * gcc.c-torture/compile/20151204.c: Skip if pdp11

[PATCH, committed] fix a number of pdp11 target test failures

2018-05-25 Thread Paul Koning
Sorry, it's been a while, previous message was incomplete. This fixes a number of test failures due to test cases that are too large for pdp11 and should be skipped. Also one test case that asks for alignment larger than what pdp11 supports. paul 2018-05-25 Paul K

[PATCH, committed] fix ICE in test suite on pdp11

2018-05-27 Thread Paul Koning
This cures an ICE caused by a wrong pattern that produces invalid register references. gcc/ChangeLog 2018-05-27 Paul Koning * config/pdp11/pdp11.md (truncsihi2): Remove. Index: config/pdp11/pdp11.md === --- config

[PATCH] add udivhi3, umodhi3 functions to libgcc

2018-05-29 Thread Paul Koning
05-29 Paul Koning * udivhi3.c: New file. * udivmodhi4.c: New file. * config/pdp11/t-pdp11 (LIB2ADD): Add the above files. Index: config/pdp11/t-pdp11 === --- config/pdp11/t-pdp11(revision 260806) +++ c

Re: [PATCH] add udivhi3, umodhi3 functions to libgcc

2018-05-29 Thread Paul Koning
> On May 29, 2018, at 4:17 PM, Jakub Jelinek wrote: > > On Tue, May 29, 2018 at 03:01:20PM -0400, Paul Koning wrote: >> +short udivmodhi4 (); > > We do want real prototypes, not K&R declarations. Fixed. I had copied that from the SImode file. > >>

Re: [PATCH] add udivhi3, umodhi3 functions to libgcc

2018-06-01 Thread Paul Koning
> On Jun 1, 2018, at 5:04 PM, Joseph Myers wrote: > > On Tue, 29 May 2018, Paul Koning wrote: > >> +unsigned short udivmodhi4(unsigned short, unsigned short, int); > > libgcc should not have any such non-static functions in the user > namespace; they should all

[PATCH] Avoid optimizing memory references with side effects in compare-elim.c

2018-06-05 Thread Paul Koning
constraint with the side effect omitted; I may try for that later on if that is still supported by the constraint machinery.) Tested against my in-progress CCmode pdp11 target. Ok to commit? paul gcc/ChangeLog: 2018-06-05 Paul Koning * compare-elim.c (addr_side_effect_check): New

[PATCH] Avoid optimizing memory references with side effects in compare-elim.c

2018-06-05 Thread Paul Koning
generate a similar RTL suitable for a matching constraint with the side effect omitted; I may try for that later on if that is still supported by the constraint machinery.) Tested against my in-progress CCmode pdp11 target. Ok to commit? paul gcc/ChangeLog: 2018-06-05 Paul Koning

Re: [PATCH] Avoid optimizing memory references with side effects in compare-elim.c

2018-06-05 Thread Paul Koning
> On Jun 5, 2018, at 3:54 PM, Richard Sandiford > wrote: > > Paul Koning writes: >> This fixes an ICE if post-reload compare elimination is done and the >> target supports post_inc and similar modes, as pdp11 does. The ICE is >> caused by a generated PARALLEL

Re: [PATCH] Avoid optimizing memory references with side effects in compare-elim.c

2018-06-06 Thread Paul Koning
> On Jun 6, 2018, at 11:53 AM, Eric Botcazou wrote: > >> That simplifies the patch, which now looks like this. Ok for trunk? >> >> paul >> >> gcc/ChangeLog: >> >> 2018-06-05 Paul Koning >> >> * compare-elim.c

Re: [PATCH] Avoid optimizing memory references with side effects in compare-elim.c

2018-06-07 Thread Paul Koning
> On Jun 7, 2018, at 11:37 AM, Eric Botcazou wrote: > >> Here is a corrected version. Ok with this change? >> >> paul >> >> gcc/ChangeLog: >> >> 2018-06-06 Paul Koning >> >> * compare-elim.c (try_merge_compare): D

Re: [PATCH] Come up with Deprecated option flag.

2018-06-11 Thread Paul Koning
> On Jun 8, 2018, at 7:09 AM, Martin Liška wrote: > > Hi. > > First follow-up MPX removal patch comes up with Deprecated option flag. > That prints warning for options that have no effect: Should this be mentioned in the internals manual (section 8.2)? paul

[PATCH, committed] fix pdp11 target test failures

2018-06-22 Thread Paul Koning
This patch fixes a number of test suite failures caused by data too large for the address space or alignment larger than supported by this target. paul testsuite/ChangeLog: 2018-06-22 Paul Koning * gcc.c-torture/execute/builtins/lib/chk.c: Use smaller alignment if

[PATCH, committed] Convert pdp11 back end to CCmode

2018-06-27 Thread Paul Koning
with a machine that has two condition code registers. Committed. paul ChangeLog: 2018-06-27 Paul Koning * common/config/pdp11/pdp11-common.c (pdp11_handle_option): Handle mutually exclusive options. * config/pdp11/constraints.md (h): New constraint. (O

[PATCH, committed] Fix insn length in pdp11 shift patterns

2018-06-28 Thread Paul Koning
This patch corrects the setting of the "length" attributes in the pdp11 target for certain shift cases. Committed. paul ChangeLog: 2018-06-28 Paul Koning * config/pdp11/pdp11-protos.h (pdp11_shift_length): New function. * config/pdp11/pdp11.c (pdp11_sh

[PATCH, committed] Add -mgnu-asm to pdp11 target, change -mdec-asm

2018-07-01 Thread Paul Koning
. paul ChangeLog: 2018-07-01 Paul Koning * common/config/pdp11/pdp11-common.c (pdp11_handle_option): Handle -munit-asm, -mgnu-asm, -mdec-asm. * config/pdp11/pdp11-protos.h (pdp11_gen_int_label): New. (pdp11_output_labelref): New

Re: [PATCH 04/25] SPECIAL_REGNO_P

2018-09-13 Thread Paul Koning
> On Sep 13, 2018, at 10:08 AM, Andrew Stubbs wrote: > > On 13/09/18 11:01, Andrew Stubbs wrote: >> The assert is caused because the def-use chains indicate that SCC conflicts >> with itself. I suppose the question is why is it doing that, but it's >> probably do do with that being a special

Re: [PATCH 04/25] SPECIAL_REGNO_P

2018-09-13 Thread Paul Koning
> On Sep 13, 2018, at 10:39 AM, Andrew Stubbs wrote: > > On 13/09/18 15:16, Paul Koning wrote: >> If you don't have machine operations that add without messing with >> condition codes, wouldn't it make sense to omit the definition of the >> add-pointer pa

Re: [PATCH 04/25] SPECIAL_REGNO_P

2018-09-13 Thread Paul Koning
> On Sep 13, 2018, at 10:58 AM, Andrew Stubbs wrote: > > On 13/09/18 15:49, Paul Koning wrote: >> It's ambiguous, because the last sentence of that paragraph says "addm3 is >> used if addptrm3 is not defined." > > I didn't read that as ambigu

Re: [PATCH] middle-end/81035: Mention that noreturn suppresses tail call optimization

2018-09-21 Thread Paul Koning
> On Sep 21, 2018, at 2:17 PM, Florian Weimer wrote: > > * Segher Boessenkool: > >> On Fri, Sep 21, 2018 at 12:59:27PM +0200, Florian Weimer wrote: >>> 2018-09-21 Florian Weimer >>> >>> PR middle-end/81035 >>> * doc/extend.texi (Common Function Attributes): Mention that >>> no

[PATCH, pdp11] Enable LRA for pdp11

2018-10-03 Thread Paul Koning
r for pointing out that LRA requires define_memory_constraint, while the old allocator is happy when memory operands use define_constraint. Committed. paul ChangeLog: 2018-10-03 Paul Koning Enable LRA register allocator for PDP11. * config/pdp11/constraints.md (Q)

[PATCH, pdp11] remove -mfloat32, -mfloat64

2018-10-05 Thread Paul Koning
y is not a familiar thing to do on this machine. Committed. paul ChangeLog: 2018-10-05 Paul Koning * config/pdp11/pdp11.h (FLOAT_TYPE_SIZE): Always 32. * config/pdp11/pdp11.opt (mfloat32): Remove. (mfloat64): Remove. * doc/invoke.texi (pdp11 -mfloat32

[PATCH, pdp11] Fix LRA failure

2018-10-08 Thread Paul Koning
This patch fixes a failure handling block moves when the LRA register allocator is used. Committed. paul ChangeLog: 2018-10-08 Paul Koning * config/pdp11/pdp11-protos.h (output_block_move): Remove. (expand_block_move): New function. * config/pdp11/pdp11.c

[PATCH, pdp11] libgcc: remove -mfloat32

2018-10-08 Thread Paul Koning
I missed a file that needed to be updated for the removal of -mfloat32. Committed. paul ChangeLog: 2018-10-08 Paul Koning * config/pdp11/t-pdp11: Remove -mfloat32 switch. Index: config/pdp11/t-pdp11

[PATCH, libgcc] Bug fix in udivmodhi4.c

2018-12-05 Thread Paul Koning
This fixes a cut & paste oversight in udivmodhi4 (which is currently used only by the pdp11 target) reported by Stefan Kanthak. Committed as obvious. paul ChangeLog: 2018-12-05 Paul Koning * udivmodhi4.c (__udivmodhi4): Fix loop end check. Index: libgcc/udivmodh

Re: [ping] Change static chain to r11 on aarch64

2018-12-12 Thread Paul Koning
> On Dec 12, 2018, at 5:12 PM, Uecker, Martin > wrote: >> ... >> I've not seen such an alternative implementation (-fno-trampolines is >> ignored on all targets I tried), > > It was implemented for Ada. But here is a patch to also > activate it for C: > > https://gcc.gnu.org/ml/gcc-patches/2

Re: [PATCH v4][C][ADA] use function descriptors instead of trampolines in C

2018-12-18 Thread Paul Koning
> On Dec 17, 2018, at 2:23 PM, Szabolcs Nagy wrote: > > On 17/12/2018 18:22, Uecker, Martin wrote: >>> >>> ... >> >> So a thread_local static variable for storing the static >> chain? > > something like that, but the more i think about it the > harder it seems: the call site of the nested f

Re: [PATCH][ARM] Correctly set SLOW_BYTE_ACCESS

2019-09-11 Thread Paul Koning
> On Sep 11, 2019, at 11:48 AM, Wilco Dijkstra wrote: > > Contrary to all documentation, SLOW_BYTE_ACCESS simply means accessing > bitfields by their declared type, which results in better codegeneration > on practically any target. So set it correctly to 1 on Arm. If the documentation is in

Re: Deprecating cc0 (and consequently cc0 targets)

2019-09-21 Thread Paul Koning
> On Sep 20, 2019, at 1:45 PM, Jeff Law wrote: > > On 9/20/19 11:22 AM, Richard Biener wrote: >> ... >> It seems to be that for the goal to keep a target alive a variant #2 >> conversion (according to the wiki) should be closely mirror CC0 >> behavior and thus should be easier to achieve and w

[PATCH, doc] describe mode checking for doloop_end pattern

2018-10-11 Thread Paul Koning
: 2018-10-11 Paul Koning * doc/md.texi (doloop_end): Document that the pattern code may need to check operand mode. Index: doc/md.texi === --- doc/md.texi (revision 265042) +++ doc/md.texi (working copy) @@ -7619,7

[PATCH, doc] describe mode checking for doloop_end pattern

2018-10-11 Thread Paul Koning
: 2018-10-11 Paul Koning * doc/md.texi (doloop_end): Document that the pattern code may need to check operand mode. Index: md.texi === --- md.texi (revision 265042) +++ md.texi (working copy) @@ -7619,7 +7619,23

Re: [PATCH, doc] describe mode checking for doloop_end pattern

2018-10-12 Thread Paul Koning
> On Oct 11, 2018, at 11:01 PM, Jeff Law wrote: > > On 10/11/18 3:09 PM, Paul Koning wrote: >> Updated with an additional item I just debugged. >> >> Since the code that uses the doloop_end pattern does not check the operand >> mode as given in the pattern,

[PATCH, pdp11] fix problem with doloop_end

2018-10-12 Thread Paul Koning
For some newlib sources, pdp11 doloop_end was creating problems because it was handed a QImode operand when it only wants HImode. This patch cures that, and also adds addqi3 and subqi3 patterns to help with that case. Committed. paul ChangeLog: 2018-10-12 Paul Koning

Re: [PATCH] add udivhi3, umodhi3 functions to libgcc

2018-10-17 Thread Paul Koning
copied does not do that, so I corrected mine and also changed the existing code to conform to the rules. Ok for trunk? paul ChangeLog: 2018-10-17 Paul Koning * udivmodsi4.c (__udivmodsi4): Rename to conform to coding standard. * udivmod.c: Update references to

Re: [PATCH] add udivhi3, umodhi3 functions to libgcc

2018-10-18 Thread Paul Koning
> On Oct 18, 2018, at 1:18 PM, Jeff Law wrote: > > On 10/17/18 5:48 PM, Paul Koning wrote: >> This is a revision of a patch I proposed a while back, to add udivhi3 and >> umodhi3 functions to libgcc since some platforms (like pdp11) need it. The >> code is a

[PATCH] correct max alignment check in symtab.c

2018-10-22 Thread Paul Koning
GCC was hitting an ICE on some pdp11 tests due to a typo in a max alignment check. Committed as obvious. paul ChangeLog: 2018-10-22 Paul Koning * symtab.c (symtab_node::increase_alignment): Correct max alignment check. Index: symtab.c

[PATCH, testsuite]: check for weak support

2018-10-25 Thread Paul Koning
paul ChangeLog: 2018-10-25 Paul Koning * gcc.dg/tree-ssa/attr-alias.c: Skip if no weak support. Index: testsuite/gcc.dg/tree-ssa/attr-alias.c === --- testsuite/gcc.dg/tree-ssa/attr-alias.c (revision 265404)

[PATCH, doc] Fix CONST_WIDE_INT_ELT

2018-10-29 Thread Paul Koning
The description of CONST_WIDE_INT_ELT gave the macro's name as CONST_WIDE_INT_NUNITS instead. Committed as obvious. paul ChangeLog: 2018-10-29 Paul Koning * doc/rtl.texi (CONST_WIDE_INT_ELT): Give correct macro name. Index: doc/rtl

Ping: [PATCH, testsuite]: check for weak support

2018-10-30 Thread Paul Koning
Ping. Ok to commit? paul > On Oct 25, 2018, at 2:57 PM, Paul Koning wrote: > > I ran into a failures due to no weak symbol support in my target. This patch > cures that. Is it right? The test case uses "weakref" so I' not 100% sure > that checking

Re: Ping: [PATCH, testsuite]: check for weak support

2018-10-30 Thread Paul Koning
> On Oct 30, 2018, at 10:17 AM, Jeff Law wrote: > > On 10/30/18 6:55 AM, Paul Koning wrote: >> Ping. Ok to commit? >> >> paul >> >>> On Oct 25, 2018, at 2:57 PM, Paul Koning wrote: >>> >>> I ran into a failures due to no w

[PATCH] libgcc build fix for pdp11

2018-11-01 Thread Paul Koning
This fixes some test suite failures due to a missing arithmetic support routine. Committed. paul ChangeLog: 2018-11-01 Paul Koning * config/pdp11/t-pdp11 (LIB2ADD): Add divmod.c. (HOST_LIBGCC2_CFLAGS): Change to optimize for size. Index: config/pdp11/t-pdp11

[PATCH, testsuite] test case fixes for pdp11

2018-11-01 Thread Paul Koning
18-11-01 Paul Koning * gcc.c-torture/execute/20010904-1.c: Align 2 if pdp11. * gcc.c-torture/execute/20010904-2.c: Ditto. * c-c++-common/builtin-arith-overflow-2.c: Skip if pdp11. * gcc.dg/Walloc-size-larger-than-4.c: Ditto. * gcc.dg/Walloc-size-larger-tha

[PATCH, testsuite] ignore some "conflicting types for built-in" messages

2018-11-01 Thread Paul Koning
e the test cases where these occur are not looking for the message but are testing some other issue, so the message is not relevant. The attached patch adds dg-prune-output directives to do so. Ok for trunk? paul ChangeLog: 2018-11-01 Paul Koning * gcc.dg/Walloca-16

[PATCH, testsuite] add "inf" target attribute

2018-11-01 Thread Paul Koning
ched patch implements this. Ok for trunk? paul ChangeLog: 2018-11-01 Paul Koning * doc/sourcebuild.texi (target attributes): Document new "inf" effective target keyword. Index: doc/sourcebuild.texi ===

Re: [PATCH, testsuite] add "inf" target attribute

2018-11-01 Thread Paul Koning
> On Nov 1, 2018, at 4:52 PM, Joseph Myers wrote: > > On Thu, 1 Nov 2018, Paul Koning wrote: > >> +@item inf >> +Target supports floating point infinite (@code{inf}). >> @end table > > Do you mean supports infinity for type double? (That's what th

Re: [PATCH, testsuite] test case fixes for pdp11

2018-11-02 Thread Paul Koning
> On Nov 2, 2018, at 3:19 PM, Rainer Orth wrote: > > Hi Paul, > >> This patch fixes a number of test case failures on pdp11. Some are too >> large for the address space, some have dependencies on the float format that >> don't match the DEC format, some add pdp11 to the targets that expect

Re: [PATCH, testsuite] ignore some "conflicting types for built-in" messages

2018-11-05 Thread Paul Koning
> On Nov 3, 2018, at 10:12 PM, Jeff Law wrote: > > On 11/1/18 1:13 PM, Paul Koning wrote: >> A number of test cases contain declarations like: >> void *memcpy(); >> which currently are silently accepted on most platforms but not on all; >> pdp11 (and

Re: [PATCH, testsuite] ignore some "conflicting types for built-in" messages

2018-11-05 Thread Paul Koning
> On Nov 5, 2018, at 11:45 AM, Jeff Law wrote: > >>> ... >> >> I can do that, but I'm wondering if some systems have different prototypes >> than the C standard calls for so I'd end up breaking those.I wouldn't worry >> about those. I think the bigger question (thanks > Martin) is whether

Re: PR83750: CSE erf/erfc pair

2018-11-05 Thread Paul Koning
> On Nov 5, 2018, at 1:48 PM, Michael Matz wrote: > > Hi, > > On Mon, 5 Nov 2018, Jeff Law wrote: > Don't we have a flag specific to honoring nans? Would that be better to use than flag_unsafe_math_optimizations? As Uli mentioned, there's >>> >>> That's only relevant for

Re: [PATCH, testsuite] add "inf" target attribute

2018-11-05 Thread Paul Koning
> On Nov 4, 2018, at 2:33 PM, Jeff Law wrote: > > On 11/1/18 1:30 PM, Paul Koning wrote: >> A number of test cases fail on pdp11 because they use the "inf" float value >> which does not exist on that target (nor on VAX). Rainer Orth and Joseph >> M

[PATCH, pdp11] Bugfixes from test suite

2018-11-08 Thread Paul Koning
This patch corrects a large number of test suite failures. I'm now down to about 1100 failures out of over 60k total, from at least 4000 before. Committed. paul ChangeLog: 2018-11-08 Paul Koning * config/pdp11/constraints.md: Add "Z" series cons

Ping: [PATCH, testsuite] ignore some "conflicting types for built-in" messages

2018-11-09 Thread Paul Koning
Ping. I'd like to commit this. The discussion seems to have ended up with the conclusion that this is a reasonable approach. paul > On Nov 1, 2018, at 3:13 PM, Paul Koning wrote: > > A number of test cases contain declarations like: > void *memcpy(); > which cu

Re: [PATCH 1/7][MSP430][TESTSUITE] Tweak dg-directives for msp430-elf

2018-11-14 Thread Paul Koning
> On Nov 14, 2018, at 10:44 AM, Jozef Lawrynowicz > wrote: > > Patch 1 tweaks dg directives in tests specifically for msp430. Many of > these are extensions to existing target selectors in dg directives. > > <0001-TESTSUITE-MSP430-Tweak-dg-directives-for-msp430-elf.patch> For pr41779.c, you

Re: [PATCH 1/7][MSP430][TESTSUITE] Tweak dg-directives for msp430-elf

2018-11-14 Thread Paul Koning
> On Nov 14, 2018, at 1:00 PM, Jozef Lawrynowicz > wrote: > > On 14/11/2018 16:54, Andreas Schwab wrote: >> On Nov 14 2018, Jozef Lawrynowicz wrote: >> >>> diff --git a/gcc/testsuite/c-c++-common/torture/builtin-arith-overflow-10.c >>> b/gcc/testsuite/c-c++-common/torture/builtin-arith-ove

Re: [PATCH 1/7][MSP430][TESTSUITE] Tweak dg-directives for msp430-elf

2018-11-15 Thread Paul Koning
> On Nov 14, 2018, at 5:19 PM, Jozef Lawrynowicz > wrote: > > On Wed, 14 Nov 2018 11:30:39 -0500 > Paul Koning wrote: > >>> On Nov 14, 2018, at 10:44 AM, Jozef Lawrynowicz >>> wrote: >>> >>> Patch 1 tweaks dg directives in tests spe

[PATCH, testsuite] indicate no "weak" support in pdp11

2018-11-19 Thread Paul Koning
gure it's best to ask for approval. Ok for trunk? paul testsuite/ChangeLog: 2018-11-19 Paul Koning * lib/target-supports.exp (check_weak_available): Return "no" for pdp11. Index: lib/target-supports.exp =

Re: [PATCH, testsuite] indicate no "weak" support in pdp11

2018-11-19 Thread Paul Koning
> On Nov 19, 2018, at 5:20 PM, Jeff Law wrote: > > On 11/19/18 3:18 PM, Paul Koning wrote: >> This patch changes check_weak_available to report that pdp11 does not >> support "weak". A number of test case failures are caused by attempts to >> use weak

[PATCH, testsuite] Fix pdp11 test failures

2018-11-19 Thread Paul Koning
test. Committed. paul testsuite/ChangeLog: 2018-11-19 Paul Koning * gcc.c-torture/execute/align-3.c: Skip if pdp11. * gcc.c-torture/execute/pr23467.c: Ditto. * gcc.c-torture/execute/pr36093.c: Ditto. * gcc.c-torture/execute/pr43783.c: Ditto

[PATCH, pdp11] Fix 64 bit divide

2018-11-25 Thread Paul Koning
This fixes a number of testsuite failures in pdp11. Committed. paul ChangeLog: 2018-11-25 Paul Koning * config/pdp11/pdp11.h (TARGET_HAS_NO_HW_DIVIDE): Define. Index: config/pdp11/pdp11.h === --- config/pdp11

Re: [PATCH] PR880088 Enable -Wtrampolines for no-exec-stack targets with -Wall.

2018-11-26 Thread Paul Koning
> On Nov 26, 2018, at 4:13 AM, Mark Wielaard wrote: > > With -Wtrampolines a warning is produced whenever gcc generates executable > code on the stack at runtime to support taking a nested function address > that is used to call the nested function indirectly when it needs to access > any vari

  1   2   >