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

[LoongArch64] Fix some bugs in the test sets 'System.Runtime.Tests' and 'System.Linq.Expressions.Tests' for loongarch64. #86435

Merged
merged 3 commits into from
May 22, 2023

Conversation

LuckyXu-HF
Copy link
Contributor

@LuckyXu-HF LuckyXu-HF commented May 18, 2023

This PR is part of the issue #69705 to amend the LA's port.

Fix some bugs in the test sets 'System.Runtime.Tests' and 'System.Linq.Expressions.Tests' for loongarch64:

  • fix the error of the Truncate/Saturate convert from Float/Double NAN to UINT32/UINTPTR.

  • fix some random failed testcases in System.Runtime.Tests.

  • fix the RangeCheck bug of the TYP_INT case in CodeGen::genRangeCheck().

@ghost ghost added the community-contribution Indicates that the PR has been added by a community member label May 18, 2023
@dotnet-issue-labeler dotnet-issue-labeler bot added the area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI label May 18, 2023
@ghost
Copy link

ghost commented May 18, 2023

Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch
See info in area-owners.md if you want to be subscribed.

Issue Details
  • fix the error of the Truncate/Saturate convert from Float/Double NAN to UINT32/UINTPTR.
  • fix some random failed testcases in System.Runtime.Tests.
Author: LuckyXu-HF
Assignees: -
Labels:

area-CodeGen-coreclr, community-contribution

Milestone: -

@LuckyXu-HF
Copy link
Contributor Author

LuckyXu-HF commented May 18, 2023

Based on this commit, the libraries test sets "System.Runtime.Tests" and "System.Linq.Expressions.Tests" has been all passed on loongarch64.

----- start 2023年 05月 18日 星期四 20:13:42 CST =============== To repro directly: =====================================================
pushd ./libs-testcase-release-v8.0.0-preview.4.23259.5/System.Runtime.Tests/Release/net8.0-unix
/home/xuliangyu/libs-8-tests/net8.0-linux-Release-LA64-v8.0.0-preview.4.23259.5/dotnet exec --runtimeconfig System.Runtime.Tests.runtimeconfig.json --depsfile System.Runtime.Tests.deps.json xunit.console.dll System.Runtime.Tests.dll -xml testResults.xml -nologo -notrait category=AdditionalTimezoneChecks -notrait category=OuterLoop -notrait category=failing 
popd
===========================================================================================================
~/libs-8-tests/libs-testcase-release-v8.0.0-preview.4.23259.5/System.Runtime.Tests/Release/net8.0-unix ~/libs-8-tests
  Discovering: System.Runtime.Tests (method display = ClassAndMethod, method display options = None)
  Discovered:  System.Runtime.Tests (found 9163 of 9210 test cases)
  Starting:    System.Runtime.Tests (parallel test collections = on, max threads = 4)
    System.Tests.DateTimeOffsetTests.ToLocalTime_MinValue [SKIP]
      Condition(s) not met: "IsMinValueNegativeLocalOffset"
    System.Tests.StringTests.IndexOf_SingleLetter(s: "Hello", target: '\0', startIndex: 0, count: 5, expected: -1) [SKIP]
      Target \0 is not supported in ICU
    System.Tests.DateTimeOffsetTests.ToLocalTime_Ambiguous [SKIP]
      Condition(s) not met: "IsPacificTime"
    System.Tests.StringTests.LastIndexOf_NullInStrings [SKIP]
      Condition(s) not met: "IsNlsGlobalization"
    System.Tests.StringTests.IndexOf_NullInStrings [SKIP]
      Condition(s) not met: "IsNlsGlobalization"
    System.Tests.StringTests.EndsWith_NullInStrings_NonOrdinal [SKIP]
      Condition(s) not met: "IsNlsGlobalization"
    System.Tests.StringTests.StartsWith_NullInStrings_NonOrdinal [SKIP]
      Condition(s) not met: "IsNlsGlobalization"
    System.Tests.StringTests.EndsWith_NullInStrings [SKIP]
      Condition(s) not met: "IsNlsGlobalization"
    System.Tests.StringTests.StartsWith_NullInStrings [SKIP]
      Condition(s) not met: "IsNlsGlobalization"
    System.Tests.ArgIteratorTests.ArgIterator_GetNextArgType [SKIP]
      Condition(s) not met: "IsArgIteratorSupported"
    System.Tests.ArgIteratorTests.ArgIterator_GetRemainingCount_GetNextArg [SKIP]
      Condition(s) not met: "IsArgIteratorSupported"
    System.Tests.TimeZoneInfoTests.UnsupportedImplicitConversionTest [SKIP]
      Condition(s) not met: "DoesNotSupportIanaNamesConversion"
    System.Runtime.Tests.JitInfoTests.JitInfoIsNotPopulated [SKIP]
      Condition(s) not met: "IsMonoAOT"
  Finished:    System.Runtime.Tests
=== TEST EXECUTION SUMMARY ===
   System.Runtime.Tests  Total: 58145, Errors: 0, Failed: 0, Skipped: 13, Time: 42.250s
~/libs-8-tests
----- end 2023年 05月 18日 星期四 20:14:28 CST ----- exit code 0 ----------------------------------------------------------
exit code 0 means Exited Successfully
2023年 05月 22日 星期一 10:20:18 CST
Testing: @@@@@ home/xuliangyu/libs-8-tests/libs-testcase-release-v8.0.0-preview.4.23259.5/System.Linq.Expressions.Tests/Release/net8.0 @@@@@
  Discovering: System.Linq.Expressions.Tests (method display = ClassAndMethod, method display options = None)
  Discovered:  System.Linq.Expressions.Tests (found 5665 of 5672 test cases)
  Starting:    System.Linq.Expressions.Tests (parallel test collections = on, max threads = 4)
  Finished:    System.Linq.Expressions.Tests
=== TEST EXECUTION SUMMARY ===
   System.Linq.Expressions.Tests  Total: 35113, Errors: 0, Failed: 0, Skipped: 0, Time: 47.403s

…to UINT32/UINT64.

* fix some random failed testcases in System.Runtime.Tests.
Copy link
Contributor

@shushanhf shushanhf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, @jakobbotsch
Could you please review this PR?

Fix the RangeCheck bug of the TYP_INT case in CodeGen::genRangeCheck().
@LuckyXu-HF LuckyXu-HF changed the title [LoongArch64] Fix some bugs in the test set 'System.Runtime.Tests' for loongarch64. [LoongArch64] Fix some bugs in the test set 'System.Runtime.Tests' and 'System.Linq.Expressions.Tests' for loongarch64. May 22, 2023
@LuckyXu-HF
Copy link
Contributor Author

LuckyXu-HF commented May 22, 2023

Hi, @jakobbotsch
This PR is mainly fix some bugs in the test sets 'System.Runtime.Tests' and 'System.Linq.Expressions.Tests' for LA64.
Could you please review this PR?
Thanks.

@LuckyXu-HF LuckyXu-HF changed the title [LoongArch64] Fix some bugs in the test set 'System.Runtime.Tests' and 'System.Linq.Expressions.Tests' for loongarch64. [LoongArch64] Fix some bugs in the test sets 'System.Runtime.Tests' and 'System.Linq.Expressions.Tests' for loongarch64. May 22, 2023
@JulieLeeMSFT JulieLeeMSFT requested a review from jakobbotsch May 22, 2023 16:50
@jakobbotsch jakobbotsch merged commit b380e2c into dotnet:main May 22, 2023
@LuckyXu-HF LuckyXu-HF deleted the main-temp branch May 24, 2023 03:39
@ghost ghost locked as resolved and limited conversation to collaborators Jun 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
arch-loongarch64 area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI community-contribution Indicates that the PR has been added by a community member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants