Skip to content

Conversation

BruceForstall
Copy link
Contributor

The on-demand Azure module load was refactored to jitutil.py,
but that requires some cross-module importing to work. Do the
minimal required to make this work.

The on-demand Azure module load was refactored to jitutil.py,
but that requires some cross-module importing to work. Do the
minimal required to make this work.
@ghost ghost added the area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI label Nov 16, 2021
@ghost
Copy link

ghost commented Nov 16, 2021

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

Issue Details

The on-demand Azure module load was refactored to jitutil.py,
but that requires some cross-module importing to work. Do the
minimal required to make this work.

Author: BruceForstall
Assignees: -
Labels:

area-CodeGen-coreclr

Milestone: -

@BruceForstall
Copy link
Contributor Author

@AndyAyersMS @echesakovMSFT @dotnet/jit-contrib PTAL

This should fix the SPMI collection pipeline, which failed to upload anything in this weekend's run.

Interestingly, I always need to force using a 32-bit Python when using the Azure stuff (like superpmi.py upload), e.g., py -3-32 .... Seems like the azure modules are 32-bit only, or maybe I installed the 32-bit versions?

Copy link
Member

@AndyAyersMS AndyAyersMS left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this.

@AndyAyersMS
Copy link
Member

Seems like the azure modules are 32-bit only

I think that may be the case. I had to uninstall 64 bit oython to get anywhere when I first ran into this a while back.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants