Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[RISCV] add missing xcvmem in RISCVISAInfo #89

Merged
merged 1 commit into from
Dec 12, 2023

Conversation

realqhc
Copy link
Contributor

@realqhc realqhc commented Nov 10, 2023

We still have a problem in cv-mem here!

.....

  • cv-mem-lb-compile-1.c
    • cv.lb\t(?:t[0-6]|a[0-7]|s[1-11]), ((?:t[0-6]|a[0-7]|s[1-11])), 1
  • cv-mem-lb-compile-2.c
    • cv.lb\t(?:t[0-6]|a[0-7]|s[1-11]), ((?:t[0-6]|a[0-7]|s[1-11])), (?:t[0-6]|a[0-7]|s[1-11])
  • cv-mem-lb-compile-3.c
  • cv-mem-lbu-compile-1.c
    • cv.lbu\t(?:t[0-6]|a[0-7]|s[1-11]), ((?:t[0-6]|a[0-7]|s[1-11])), 1
  • cv-mem-lbu-compile-2.c
    • cv.lbu\t(?:t[0-6]|a[0-7]|s[1-11]), ((?:t[0-6]|a[0-7]|s[1-11])), (?:t[0-6]|a[0-7]|s[1-11])
  • cv-mem-lbu-compile-3.c
  • cv-mem-lh-compile-1.c
    • cv.lh\t(?:t[0-6]|a[0-7]|s[1-11]), ((?:t[0-6]|a[0-7]|s[1-11])), 2
  • cv-mem-lh-compile-2.c
    • cv.lh\t(?:t[0-6]|a[0-7]|s[1-11]), ((?:t[0-6]|a[0-7]|s[1-11])), (?:t[0-6]|a[0-7]|s[1-11])
  • cv-mem-lh-compile-3.c
  • cv-mem-lhu-compile-1.c
    • cv.lhu\t(?:t[0-6]|a[0-7]|s[1-11]), ((?:t[0-6]|a[0-7]|s[1-11])), 2
  • cv-mem-lhu-compile-2.c
    • cv.lhu\t(?:t[0-6]|a[0-7]|s[1-11]), ((?:t[0-6]|a[0-7]|s[1-11])), (?:t[0-6]|a[0-7]|s[1-11])
  • cv-mem-lhu-compile-3.c
  • cv-mem-lw-compile-1.c
    • cv.lw\t(?:t[0-6]|a[0-7]|s[1-11]), ((?:t[0-6]|a[0-7]|s[1-11])), 4
  • cv-mem-lw-compile-2.c
    • cv.lw\t(?:t[0-6]|a[0-7]|s[1-11]), ((?:t[0-6]|a[0-7]|s[1-11])), (?:t[0-6]|a[0-7]|s[1-11])
  • cv-mem-lw-compile-3.c
  • cv-mem-lw-compile-4.c
    • None
  • cv-mem-sb-compile-1.c
    • cv.sb\t(?:t[0-6]|a[0-7]|s[1-11]), ((?:t[0-6]|a[0-7]|s[1-11])), 1
  • cv-mem-sb-compile-2.c
    • cv.sb\t(?:t[0-6]|a[0-7]|s[1-11]), ((?:t[0-6]|a[0-7]|s[1-11])), (?:t[0-6]|a[0-7]|s[1-11])
  • cv-mem-sb-compile-3.c
  • cv-mem-sh-compile-1.c
    • cv.sh\t(?:t[0-6]|a[0-7]|s[1-11]), ((?:t[0-6]|a[0-7]|s[1-11])), 2
  • cv-mem-sh-compile-2.c
    • cv.sh\t(?:t[0-6]|a[0-7]|s[1-11]), ((?:t[0-6]|a[0-7]|s[1-11])), (?:t[0-6]|a[0-7]|s[1-11])
  • cv-mem-sh-compile-3.c
  • cv-mem-sw-compile-1.c
    • cv.sw\t(?:t[0-6]|a[0-7]|s[1-11]), ((?:t[0-6]|a[0-7]|s[1-11])), 4
  • cv-mem-sw-compile-2.c
    • cv.sw\t(?:t[0-6]|a[0-7]|s[1-11]), ((?:t[0-6]|a[0-7]|s[1-11])), (?:t[0-6]|a[0-7]|s[1-11])
  • cv-mem-sw-compile-3.c
  • cv-mem-sw-compile-4.c
    • None
      .....

@realqhc realqhc closed this Nov 13, 2023
@realqhc realqhc reopened this Nov 13, 2023
@MaryBennett
Copy link

Looks good to merge. Reviewed in the meeting 24th November.

@realqhc
Copy link
Contributor Author

realqhc commented Nov 28, 2023

- [x] cv-mem-lb-compile-3.c
- [x] cv-mem-lbu-compile-3.c
- [x] cv-mem-lh-compile-3.c
- [x] cv-mem-lhu-compile-3.c
- [x] cv-mem-lw-compile-3.c
- [x] cv-mem-lw-compile-4.c
- [x] cv-mem-sb-compile-3.c
- [x] cv-mem-sh-compile-3.c
- [x] cv-mem-sw-compile-3.c
- [x] cv-mem-sw-compile-4.c

after I update the gcc tests the number of failures are reduced

@PaoloS02
Copy link
Contributor

Good progress! Are there still gcc failures to fix? Could you update the full list at the top if it changed?
Also, less important, does the development branch at the base of this (and PR #90 ) need to be updated? I see there is still that MacOS failure.

@PaoloS02 PaoloS02 merged commit 48d6907 into openhwgroup:development Dec 12, 2023
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants