[Bug target/79242] ICE in simplify_subreg, at simplify-rtx.c:6029

2017-11-22 Thread jozefl.gcc at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79242 Jozef Lawrynowicz changed: What|Removed |Added CC||jozefl.gcc at gmail dot com

[Bug target/79242] ICE in simplify_subreg, at simplify-rtx.c:6029

2018-01-12 Thread jozefl.gcc at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79242 --- Comment #8 from Jozef Lawrynowicz --- Proposed patch: https://gcc.gnu.org/ml/gcc-patches/2018-01/msg01108.html

[Bug target/84406] New: [8 Regression][MSP430] ICE on valid code in find_widening_optab_handler_and_mode, at optabs-query.c:476

2018-02-15 Thread jozefl.gcc at gmail dot com
Status: UNCONFIRMED Severity: normal Priority: P3 Component: target Assignee: unassigned at gcc dot gnu.org Reporter: jozefl.gcc at gmail dot com Target Milestone: --- Trunk currently fails to build for the msp430-elf target due to an ICE

[Bug target/79242] [7 Regression] ICE in simplify_subreg, at simplify-rtx.c:6029

2018-02-15 Thread jozefl.gcc at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79242 --- Comment #11 from Jozef Lawrynowicz --- (In reply to Eric Gallager from comment #10) > (In reply to Jeffrey A. Law from comment #9) > > Author: law > > Date: Wed Feb 14 07:21:11 2018 > > New Revision: 257653 > > > > URL:

[Bug target/84406] [8 Regression][MSP430] ICE on valid code in find_widening_optab_handler_and_mode, at optabs-query.c:476

2018-02-15 Thread jozefl.gcc at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84406 --- Comment #3 from Jozef Lawrynowicz --- Created attachment 43433 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43433=edit define {u,}mulhipsi3 insns I applied the patch but trunk still ICEs. After defining some insns for "umulhipsi3"

[Bug target/84406] [8 Regression][MSP430] ICE on valid code in find_widening_optab_handler_and_mode, at optabs-query.c:476

2018-02-15 Thread jozefl.gcc at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84406 --- Comment #12 from Jozef Lawrynowicz --- (In reply to rsand...@gcc.gnu.org from comment #11) > Created attachment 43437 [details] > Possible patch v2 > > This time for real. Thanks, patch looks good to me, and the original testcase does

[Bug rtl-optimization/90032] New: [MSP430] reload uses wrong stack slot for variable after setjmp/longjmp

2019-04-09 Thread jozefl.gcc at gmail dot com
Priority: P3 Component: rtl-optimization Assignee: unassigned at gcc dot gnu.org Reporter: jozefl.gcc at gmail dot com Target Milestone: --- Created attachment 46118 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=46118=edit tester.i gcc.dg/tort

[Bug rtl-optimization/90032] [MSP430] reload uses wrong stack slot for variable after setjmp/longjmp

2019-04-09 Thread jozefl.gcc at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90032 --- Comment #2 from Jozef Lawrynowicz --- Created attachment 46120 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=46120=edit tester.i ira dump

[Bug rtl-optimization/90032] [MSP430] reload uses wrong stack slot for variable after setjmp/longjmp

2019-04-09 Thread jozefl.gcc at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90032 --- Comment #3 from Jozef Lawrynowicz --- Created attachment 46121 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=46121=edit tester.i reload dump

[Bug rtl-optimization/90032] [MSP430] reload uses wrong stack slot for variable after setjmp/longjmp

2019-04-09 Thread jozefl.gcc at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90032 --- Comment #4 from Jozef Lawrynowicz --- Created attachment 46122 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=46122=edit tester.s

[Bug rtl-optimization/90032] [MSP430] reload uses wrong stack slot for variable after setjmp/longjmp

2019-04-09 Thread jozefl.gcc at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90032 Jozef Lawrynowicz changed: What|Removed |Added Attachment #46118|0 |1 is obsolete|

[Bug testsuite/90812] Tests misuse "dg-require-effective-target int32plus" to check for 64-bit integer support

2019-06-13 Thread jozefl.gcc at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90812 --- Comment #3 from Jozef Lawrynowicz --- (In reply to Richard Biener from comment #2) > I think most tests like this end up using 'long long' and use > __SIZEOF_LONG_LONG__ to guard code. There's a dejagnu effective target for > long long

[Bug testsuite/90812] New: Tests misuse "dg-require-effective-target int32plus" to check for 64-bit integer support

2019-06-10 Thread jozefl.gcc at gmail dot com
NCONFIRMED Severity: normal Priority: P3 Component: testsuite Assignee: unassigned at gcc dot gnu.org Reporter: jozefl.gcc at gmail dot com Target Milestone: --- Some tests which require 64-bit integer support via __INT64_TYPE__ use the int32plus effect

[Bug testsuite/90812] Tests misuse "dg-require-effective-target int32plus" to check for 64-bit integer support

2019-06-10 Thread jozefl.gcc at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90812 --- Comment #1 from Jozef Lawrynowicz --- To check for 64-bit integer support, it seems that the "stdint_types" effective target keyword should be sufficient, although I'm not sure if it might be possible for a target to provide __INT64_TYPE__

[Bug target/90175] ambiguous wording "critical attribute" in diagnostic

2019-08-28 Thread jozefl.gcc at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90175 Jozef Lawrynowicz changed: What|Removed |Added CC||jozefl.gcc at gmail dot com

[Bug target/91306] [MSP430] libgcc/crtstuff.c: Alignment of frame_dummy .init_array entry is too big

2019-08-26 Thread jozefl.gcc at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91306 Jozef Lawrynowicz changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug libgcc/91306] New: [MSP430] libgcc/crtstuff.c: Alignment of frame_dummy .init_array entry is too big

2019-07-31 Thread jozefl.gcc at gmail dot com
: normal Priority: P3 Component: libgcc Assignee: unassigned at gcc dot gnu.org Reporter: jozefl.gcc at gmail dot com Target Milestone: --- In libgcc/crtstuff.c, an alignment of "sizeof (void *)" (i.e. pointer size) is enforced for the f

[Bug target/91306] [MSP430] libgcc/crtstuff.c: Alignment of frame_dummy .init_array entry is too big

2019-08-01 Thread jozefl.gcc at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91306 --- Comment #3 from Jozef Lawrynowicz --- (In reply to Andrew Pinski from comment #2) > If that is true, then maybe alignof should be used instead of sizeof. > Can you try alignof (__alignof__)? Thanks, __alignof__ works for me. Below change

[Bug target/91306] [MSP430] libgcc/crtstuff.c: Alignment of frame_dummy .init_array entry is too big

2019-08-08 Thread jozefl.gcc at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91306 --- Comment #4 from Jozef Lawrynowicz --- Should I submit a patch which changes uses of sizeof in alignment attributes to __alignof__? Or are you working on it?

[Bug rtl-optimization/91865] New: Combine misses opportunity to remove (sign_extend (zero_extend)) before searching for insn patterns

2019-09-23 Thread jozefl.gcc at gmail dot com
Severity: normal Priority: P3 Component: rtl-optimization Assignee: unassigned at gcc dot gnu.org Reporter: jozefl.gcc at gmail dot com Target Milestone: --- Created attachment 46913 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=46913=e

[Bug rtl-optimization/91865] Combine misses opportunity to remove (sign_extend (zero_extend)) before searching for insn patterns

2019-09-24 Thread jozefl.gcc at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91865 --- Comment #3 from Jozef Lawrynowicz --- Created attachment 46936 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=46936=edit msp430-extendhipsi2.diff

[Bug rtl-optimization/91865] Combine misses opportunity to remove (sign_extend (zero_extend)) before searching for insn patterns

2019-09-24 Thread jozefl.gcc at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91865 --- Comment #2 from Jozef Lawrynowicz --- A related issue can be observed if "char i" is made global instead of an argument to func. const int table[2] = {1, 2}; int foo; char i; void func(void) { foo = table[i]; } Combine combines the

[Bug target/92287] Mismatches in the calling convention for zero sized types

2019-10-30 Thread jozefl.gcc at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92287 Jozef Lawrynowicz changed: What|Removed |Added CC||jozefl.gcc at gmail dot com

[Bug target/92287] Mismatches in the calling convention for zero sized types

2019-10-30 Thread jozefl.gcc at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92287 --- Comment #3 from Jozef Lawrynowicz --- (In reply to gnzlbg from comment #2) > > I can only speak for msp430, but there's no problem with that generated > > assembly. Structures and unions are always passed by reference. > > I suppose that

[Bug target/92287] Mismatches in the calling convention for zero sized types

2019-11-01 Thread jozefl.gcc at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92287 --- Comment #10 from Jozef Lawrynowicz --- (In reply to gnzlbg from comment #9) > > @josef > > > The MSP430 ABI is here: http://www.ti.com/lit/an/slaa534/slaa534.pdf > Although confusingly that document is wrong regarding passing structures

[Bug target/70320] msp430 asm volatile does not accept lower-case register names in clobber list

2019-10-30 Thread jozefl.gcc at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70320 Jozef Lawrynowicz changed: What|Removed |Added CC||jozefl.gcc at gmail dot com

[Bug rtl-optimization/91865] Combine misses opportunity to remove (sign_extend (zero_extend)) before searching for insn patterns

2019-12-27 Thread jozefl.gcc at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91865 --- Comment #5 from Jozef Lawrynowicz --- (In reply to Segher Boessenkool from comment #4) > Created attachment 47550 [details] > simplify-rtx patch for extends of AND of hard reg > > So I did a patch to make this work (in simplify-rtx) also