[Bug target/93304] RISC-V: Function with interrupt attribute use register without save/restore at prologue/epilogue

2020-01-29 Thread cvs-commit at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93304

--- Comment #5 from CVS Commits  ---
The releases/gcc-9 branch has been updated by Kito Cheng :

https://gcc.gnu.org/g:afb84a42ad867c117d0112fbb8edd863bdc0dafe

commit r9-8191-gafb84a42ad867c117d0112fbb8edd863bdc0dafe
Author: Kito Cheng 
Date:   Fri Jan 17 19:49:15 2020 +0800

RISC-V: Disallow regrenme if the TO register never used before for
interrupt functions

gcc/ChangeLog

PR target/93304
* config/riscv/riscv-protos.h (riscv_hard_regno_rename_ok): New.
* config/riscv/riscv.c (riscv_hard_regno_rename_ok): New.
* config/riscv/riscv.h (HARD_REGNO_RENAME_OK): Defined.

gcc/testsuite/ChangeLog

PR target/93304
* gcc.target/riscv/pr93304.c: New test.

[Bug target/93304] RISC-V: Function with interrupt attribute use register without save/restore at prologue/epilogue

2020-01-21 Thread wilson at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93304

Jim Wilson  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #4 from Jim Wilson  ---
Fixed on mainline.

[Bug target/93304] RISC-V: Function with interrupt attribute use register without save/restore at prologue/epilogue

2020-01-20 Thread cvs-commit at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93304

--- Comment #3 from CVS Commits  ---
The master branch has been updated by Kito Cheng :

https://gcc.gnu.org/g:e0a5b313c1a3edfb33a28b8d8fea92e01490ebb3

commit r10-6101-ge0a5b313c1a3edfb33a28b8d8fea92e01490ebb3
Author: Kito Cheng 
Date:   Fri Jan 17 19:49:15 2020 +0800

RISC-V: Disallow regrenme if the TO register never used before for
interrupt functions

gcc/ChangeLog

PR target/93304
* config/riscv/riscv-protos.h (riscv_hard_regno_rename_ok): New.
* config/riscv/riscv.c (riscv_hard_regno_rename_ok): New.
* config/riscv/riscv.h (HARD_REGNO_RENAME_OK): Defined.

gcc/testsuite/ChangeLog

PR target/93304
* gcc.target/riscv/pr93304.c: New test.

[Bug target/93304] RISC-V: Function with interrupt attribute use register without save/restore at prologue/epilogue

2020-01-20 Thread marxin at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93304

Martin Liška  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
   Last reconfirmed||2020-01-20
 CC||marxin at gcc dot gnu.org
 Ever confirmed|0   |1

[Bug target/93304] RISC-V: Function with interrupt attribute use register without save/restore at prologue/epilogue

2020-01-17 Thread wilson at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93304

Jim Wilson  changed:

   What|Removed |Added

 CC||wilson at gcc dot gnu.org

--- Comment #2 from Jim Wilson  ---
There is a convention of using all caps for function arguments.  See for
instance the riscv_build_integer function comment.  It would be nice to
preserve this convention, but this is a very minor issue.  I usually put a
blank line between the function comment and the function, but again this is a
very minor issue.

The patch looks OK to me.

[Bug target/93304] RISC-V: Function with interrupt attribute use register without save/restore at prologue/epilogue

2020-01-17 Thread kito at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93304

--- Comment #1 from Kito Cheng  ---
Created attachment 47669
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=47669&action=edit
pr93304-v1.patch