On 10/30/23 10:02, Neal Frager wrote:
The MICROBLAZE_VERSION_COMPARE was incorrectly using strcasecmp
instead of strverscmp to check the mcpu version against feature
options. By simply changing the define to use strverscmp,
the new version 10.0 is treated correctly as a higher version
than previous versions.
Signed-off-by: Neal Frager <neal.fra...@amd.com>
Added to commit message;
Fix incorrect warning with -mcpu=10.0:
warning: '-mxl-multiply-high' can be used only with
'-mcpu=v6.00.a' or greater
---
V1->V2:
- No need to create a new microblaze specific version check
routine as strverscmp is the correct solution.
V2->V3:
- Changed mcpu define for microblaze isa testsuite examples.
V3->V4:
- Added ChangeLog
V4->V5:
- Added testsuite ChangeLog
V5->V6:
- Updated testsuite ChangeLog to include all files
---
gcc/ChangeLog | 4 ++++
gcc/config/microblaze/microblaze.cc | 2 +-
gcc/testsuite/ChangeLog | 22 +++++++++++++++++++
.../gcc.target/microblaze/isa/bshift.c | 2 +-
gcc/testsuite/gcc.target/microblaze/isa/div.c | 2 +-
.../gcc.target/microblaze/isa/fcmp1.c | 2 +-
.../gcc.target/microblaze/isa/fcmp2.c | 2 +-
.../gcc.target/microblaze/isa/fcmp3.c | 2 +-
.../gcc.target/microblaze/isa/fcmp4.c | 2 +-
.../gcc.target/microblaze/isa/fcvt.c | 2 +-
.../gcc.target/microblaze/isa/float.c | 2 +-
.../gcc.target/microblaze/isa/fsqrt.c | 2 +-
.../microblaze/isa/mul-bshift-pcmp.c | 2 +-
.../gcc.target/microblaze/isa/mul-bshift.c | 2 +-
gcc/testsuite/gcc.target/microblaze/isa/mul.c | 2 +-
.../microblaze/isa/mulh-bshift-pcmp.c | 2 +-
.../gcc.target/microblaze/isa/mulh.c | 2 +-
.../gcc.target/microblaze/isa/nofcmp.c | 2 +-
.../gcc.target/microblaze/isa/nofloat.c | 2 +-
.../gcc.target/microblaze/isa/pcmp.c | 2 +-
.../gcc.target/microblaze/isa/vanilla.c | 2 +-
.../gcc.target/microblaze/microblaze.exp | 2 +-
22 files changed, 46 insertions(+), 20 deletions(-)
Committed.
--
Michael Eager
