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

cost model debug printouts #45

Open
mppf opened this issue Jul 14, 2020 · 2 comments
Open

cost model debug printouts #45

mppf opened this issue Jul 14, 2020 · 2 comments

Comments

@mppf
Copy link

mppf commented Jul 14, 2020

a8499af changed src/analysis/costModel.cpp to turn off some debug printouts, but these seem to have reappeared in later revisions and are there in the current master branch.

Is it possible to add some sort of test to check RV is not released in this way? Or to make it an LLVM command line option?

@mppf mppf mentioned this issue Jul 14, 2020
@mppf
Copy link
Author

mppf commented Jul 14, 2020

The reason that I care about this is that I'm working on integrating RV with Chapel and the Chapel test suite will report failure if there is unexpected output. Besides which these printouts are probably not helpful to end users.

@simoll
Copy link
Member

simoll commented Jul 20, 2020

Issues like this one have been reappearing for a while.
We are planning to move to automatic testing, CI as we are maturing RV development from pure 'academic' style to a more product-like approach.

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

No branches or pull requests

2 participants