https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91347
--- Comment #18 from dave.anglin at bell dot net ---
On 2019-08-19 4:36 a.m., ebotcazou at gcc dot gnu.org wrote:
> Created attachment 46728
> --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=46728=edit
> Execution test
Works on hppa without
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91347
Eric Botcazou changed:
What|Removed |Added
Attachment #46728|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91347
--- Comment #16 from Eric Botcazou ---
> Yes. It aborts with current gcc-9.2.1 and passes with patched gcc-10.
> -fno-inline is needed.
Thanks. Let's drop the -fno-inline and put it in gcc.c-torture/execute.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91347
--- Comment #15 from dave.anglin at bell dot net ---
On 2019-08-19 4:35 a.m., ebotcazou at gcc dot gnu.org wrote:
> OK, thanks. Can you check that the testcase to be attached is a valid
> execution test for trunk when compiled with -O2
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91347
--- Comment #14 from Sven Schnelle ---
I tested the patch with my (previously broken) kernel Build, and the issue
seems to be fixed. Thanks!
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91347
--- Comment #13 from Eric Botcazou ---
Created attachment 46728
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=46728=edit
Execution test
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91347
--- Comment #12 from Eric Botcazou ---
> hppa-unknown-linux-gnu built successfully with change and there were no test
> regressions:
> https://gcc.gnu.org/ml/gcc-testresults/2019-08/msg01861.html
>
> Looks good to me.
OK, thanks. Can you
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91347
--- Comment #11 from dave.anglin at bell dot net ---
On 2019-08-16 8:17 a.m., John David Anglin wrote:
> On 2019-08-16 5:47 a.m., ebotcazou at gcc dot gnu.org wrote:
>> Created attachment 46721
>> -->
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91347
--- Comment #10 from John David Anglin ---
(In reply to Eric Botcazou from comment #6)
> This looks similar to PR target/55023.
Yes. Looking at how scan_insn() evolved since PR target/55023, it is
clear why this is a regression.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91347
--- Comment #9 from dave.anglin at bell dot net ---
On 2019-08-16 5:47 a.m., ebotcazou at gcc dot gnu.org wrote:
> Created attachment 46721
> --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=46721=edit
> Tentative fix
Testing. Looking at
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91347
--- Comment #8 from Eric Botcazou ---
Created attachment 46721
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=46721=edit
Tentative fix
Please give it a try on the PA.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91347
Eric Botcazou changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|unassigned
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91347
--- Comment #6 from Eric Botcazou ---
This looks similar to PR target/55023.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91347
Eric Botcazou changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91347
--- Comment #4 from dave.anglin at bell dot net ---
If DSE bug can't be fixed, the attached patch appears to work around the issue
by disabling
tail calls on pa when an argument is passed on stack.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91347
--- Comment #3 from John David Anglin ---
I see in dse.c:
/* Get arguments passed to CALL_INSN. Return TRUE if successful.
So far it only handles arguments passed in registers. */
static bool
get_call_args (rtx call_insn, tree fn, rtx
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91347
John David Anglin changed:
What|Removed |Added
CC||ebotcazou at gcc dot gnu.org
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91347
--- Comment #1 from John David Anglin ---
The uses on the call insn seem okay (i.e., they have mem:BLK for spec):
(call_insn/j 53 52 0 5 (parallel [
(set (reg:SI 28 %r28)
(call (mem:SI (symbol_ref/v:SI ("@number")
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91347
Richard Biener changed:
What|Removed |Added
Keywords||wrong-code
Target Milestone|---
19 matches
Mail list logo
