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.

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

[polly] Add polly-debug flag to print debug info from all parts of polly #78549

Merged
merged 17 commits into from
Mar 26, 2024

Conversation

kartcq
Copy link
Contributor

@kartcq kartcq commented Jan 18, 2024

This flag enable the user to print debug Info from all the passes and helpers inside polly at once. This will help a novice user as well to work in polly without explicitly having to know which parts of polly has actually kicked in and pass them via -debug-only.

This flag enable the user to print debug Info from all the passes and
helpers inside polly at once. This will help a novice user as well to
work in polly without explicitly having to know which parts of polly has
actually kicked in and pass them via -debug-only.

Change-Id: I6ddb03d5e7debac16413fb364afb8811139c8073
Copy link

github-actions bot commented Jan 18, 2024

✅ With the latest revision this PR passed the C/C++ code formatter.

polly/include/polly/Support/PollyDebug.inc Outdated Show resolved Hide resolved
polly/include/polly/Support/PollyDebug.inc Outdated Show resolved Hide resolved
@efriedma-quic
Copy link
Collaborator

(In case you didn't notice, latest version fails on buildbot.)

Copy link
Collaborator

@efriedma-quic efriedma-quic left a comment

Choose a reason for hiding this comment

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

LGTM

I'll wait a few days before merging in case @Meinersbur has further comments.

@kartcq
Copy link
Contributor Author

kartcq commented Mar 17, 2024

Can we merge this patch ? @efriedma-quic @Meinersbur

@efriedma-quic efriedma-quic merged commit 601d7ea into llvm:main Mar 26, 2024
4 checks passed
@kartcq kartcq deleted the polly_debug branch July 31, 2024 06:07
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