Skip to content

[MachineVerifier][NewPM] Add method to run MF through verifier. #125701

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

Merged
merged 1 commit into from
Feb 5, 2025

Conversation

cdevadas
Copy link
Collaborator

@cdevadas cdevadas commented Feb 4, 2025

No description provided.

/// \returns true if no problems were found.
bool verify(MachineFunctionAnalysisManager &MFAM,
const char *Banner = nullptr, raw_ostream *OS = nullptr,
bool AbortOnError = true) const;
Copy link
Contributor

Choose a reason for hiding this comment

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

We probably should have one that separately takes all of the analyses (I thought I had a patch to do that for llvm-reduce)

Copy link
Collaborator Author

cdevadas commented Feb 5, 2025

Merge activity

  • Feb 5, 12:58 AM EST: A user started a stack merge that includes this pull request via Graphite.
  • Feb 5, 1:13 AM EST: Graphite rebased this pull request as part of a merge.
  • Feb 5, 1:19 AM EST: Graphite rebased this pull request as part of a merge.
  • Feb 5, 1:24 AM EST: A user merged this pull request with Graphite.

Base automatically changed from users/cdevadas/move-sched-helper-funcs-out-of-passconfig to main February 5, 2025 06:11
@cdevadas cdevadas force-pushed the users/cdevadas/run-mf-through-verifier-in-npm branch from effa5e3 to 20fc888 Compare February 5, 2025 06:12
@cdevadas cdevadas force-pushed the users/cdevadas/run-mf-through-verifier-in-npm branch from 20fc888 to 4b1b71d Compare February 5, 2025 06:18
@cdevadas cdevadas merged commit 1d22318 into main Feb 5, 2025
5 of 7 checks passed
@cdevadas cdevadas deleted the users/cdevadas/run-mf-through-verifier-in-npm branch February 5, 2025 06:30
Icohedron pushed a commit to Icohedron/llvm-project that referenced this pull request Feb 11, 2025
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.

2 participants