Skip to content

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.

3 participants