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.

Sign up for GitHub

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

Jump to bottom

Add New Test to Monitor Number of Jitted Methods #90681

Closed
wants to merge 17 commits into from
Closed

Add New Test to Monitor Number of Jitted Methods #90681

wants to merge 17 commits into from

Commits on Aug 3, 2023

  1. Added my first draft for the JIT methods counter tests.

    @ivdiazsa
    ivdiazsa committed Aug 3, 2023
    Configuration menu
    Copy the full SHA
    c7c02a5 View commit details
    Browse the repository at this point in the history

Commits on Aug 11, 2023

  1. Began incorporating the HelloWorld with the test builds.

    @ivdiazsa
    ivdiazsa committed Aug 11, 2023
    Configuration menu
    Copy the full SHA
    c7df757 View commit details
    Browse the repository at this point in the history

  2. Added the Jitted Methods counting test to JIT/Performance.

    @ivdiazsa
    ivdiazsa committed Aug 11, 2023
    Configuration menu
    Copy the full SHA
    9b9bb10 View commit details
    Browse the repository at this point in the history

Commits on Aug 14, 2023

  1. Made more progress. Still doesn't work.

    @ivdiazsa
    ivdiazsa committed Aug 14, 2023
    Configuration menu
    Copy the full SHA
    1713f45 View commit details
    Browse the repository at this point in the history

Commits on Aug 16, 2023

  1. Clean up.

    @ivdiazsa
    ivdiazsa committed Aug 16, 2023
    Configuration menu
    Copy the full SHA
    e0080e9 View commit details
    Browse the repository at this point in the history

  2. Added information to the test as per Tomas' feedback.

    @ivdiazsa
    ivdiazsa committed Aug 16, 2023
    Configuration menu
    Copy the full SHA
    474d0aa View commit details
    Browse the repository at this point in the history

  3. Forgot to uncomment some code.

    @ivdiazsa
    ivdiazsa committed Aug 16, 2023
    Configuration menu
    Copy the full SHA
    908e50e View commit details
    Browse the repository at this point in the history

  4. Removed the ReadyToRun and TieredCompilation env vars as per JanK's f… 

    …eedback, and cleaned up some stuff, adding the CLRTestKind property to HelloWorld.csproj, as per Tomas' feedback.
    @ivdiazsa
    ivdiazsa committed Aug 16, 2023
    Configuration menu
    Copy the full SHA
    3707cf8 View commit details
    Browse the repository at this point in the history

  5. Fixed a couple semantic errors I overlooked in my previous commit.

    @ivdiazsa
    ivdiazsa committed Aug 16, 2023
    Configuration menu
    Copy the full SHA
    eec4acb View commit details
    Browse the repository at this point in the history

Commits on Aug 17, 2023

  1. Merge branch 'main' into jit-participants

    @ivdiazsa
    ivdiazsa committed Aug 17, 2023
    Configuration menu
    Copy the full SHA
    8fa21fb View commit details
    Browse the repository at this point in the history

  2. Made the test only run in ReadyToRun environments, and addressed some… 

    … minor fixes from received feedback.
    @ivdiazsa
    ivdiazsa committed Aug 17, 2023
    Configuration menu
    Copy the full SHA
    7d36e14 View commit details
    Browse the repository at this point in the history

  3. Replaced the 'is null' check with 'string.IsNullOrEmpty()' to play it… 

    … safer, as per Tomas' feedback.
    @ivdiazsa
    ivdiazsa committed Aug 17, 2023
    Configuration menu
    Copy the full SHA
    34b0ded View commit details
    Browse the repository at this point in the history

Commits on Aug 18, 2023

  1. Moved the JittedMethodsCountingTest to src/tests/readytorun as per Br… 

    …uce's feedback.
    @ivdiazsa
    ivdiazsa committed Aug 18, 2023
    Configuration menu
    Copy the full SHA
    1c9684c View commit details
    Browse the repository at this point in the history

Commits on Aug 22, 2023

  1. Changed app paths to be absolute instead of relative.

    @ivdiazsa
    ivdiazsa committed Aug 22, 2023
    Configuration menu
    Copy the full SHA
    6029d89 View commit details
    Browse the repository at this point in the history

Commits on Aug 23, 2023

  1. DEBUG-ONLY CHANGE: Helix just plain refuses to find the file that is … 

    …supposedly there. Added an 'ls -R' command to see its tree structure and try to figure out why it can't find the file.
    @ivdiazsa
    ivdiazsa committed Aug 23, 2023
    Configuration menu
    Copy the full SHA
    04d88e2 View commit details
    Browse the repository at this point in the history

Commits on Aug 24, 2023

  1. DEBUG-ONLY CHANGE: Helix seems to not like '&&' to concatenate comman… 

    …ds, so switched to a ';' instead.
    @ivdiazsa
    ivdiazsa committed Aug 24, 2023
    Configuration menu
    Copy the full SHA
    e3b96c8 View commit details
    Browse the repository at this point in the history

Commits on Aug 25, 2023

  1. DEBUG-ONLY CHANGE: Maybe this will print the tree?

    @ivdiazsa
    ivdiazsa committed Aug 25, 2023
    Configuration menu
    Copy the full SHA
    f300f88 View commit details
    Browse the repository at this point in the history