Skip to content

Commit

Permalink
Restructure JIT to have more extension points and more logical contro…
Browse files Browse the repository at this point in the history
…l flow (#50650)

Rather than forking the optimize and compile layers into individual
layers per optlevel, each optimize and compile layer will switch on the
requested optlevel of the module, which reduces the complexity of
tracing a module's path through the JIT. In addition, this lets us move
some of the `addModule` code to happen post-optimization, which makes
the optimization pipeline not see literal pointers except those
generated by codegen.
  • Loading branch information
pchintalapudi authored Jul 27, 2023
1 parent da19bc1 commit 943db02
Show file tree
Hide file tree
Showing 2 changed files with 220 additions and 165 deletions.
Loading

7 comments on commit 943db02

@nanosoldier
Copy link
Collaborator

Choose a reason for hiding this comment

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

Executing the daily package evaluation, I will reply here when finished:

@nanosoldier runtests(isdaily = true)

@nanosoldier
Copy link
Collaborator

Choose a reason for hiding this comment

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

The package evaluation job you requested has completed - no new issues were detected.
The full report is available.

@maleadt
Copy link
Member

Choose a reason for hiding this comment

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

Something went wrong here. Let's try again:

@nanosoldier runtests(isdaily = true)

@nanosoldier
Copy link
Collaborator

Choose a reason for hiding this comment

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

Your job failed. Consult the server logs for more details (cc @maleadt).

@maleadt
Copy link
Member

Choose a reason for hiding this comment

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

@nanosoldier runtests(isdaily = true)

@maleadt
Copy link
Member

Choose a reason for hiding this comment

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

@nanosoldier runtests(isdaily = true)

@nanosoldier
Copy link
Collaborator

Choose a reason for hiding this comment

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

The package evaluation job you requested has completed - possible new issues were detected.
The full report is available.

Please sign in to comment.