-
Notifications
You must be signed in to change notification settings - Fork 122
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
Remove the use_enzyme state from reverse mode visitor. NFC #946
Conversation
clang-tidy review says "All clean, LGTM! 👍" |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #946 +/- ##
==========================================
- Coverage 93.80% 93.80% -0.01%
==========================================
Files 55 55
Lines 7828 7826 -2
==========================================
- Hits 7343 7341 -2
Misses 485 485
|
6a069f2
to
22c027e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
@@ -63,7 +63,7 @@ namespace clad { | |||
std::vector<Stmts> m_LoopBlock; | |||
unsigned outputArrayCursor = 0; | |||
unsigned numParams = 0; | |||
bool use_enzyme = false; | |||
bool isVectorValued = false; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
warning: invalid case style for protected member 'isVectorValued' [readability-identifier-naming]
bool isVectorValued = false; | |
bool m_isVectorValued = false; |
@@ -63,7 +63,7 @@ | |||
std::vector<Stmts> m_LoopBlock; | |||
unsigned outputArrayCursor = 0; | |||
unsigned numParams = 0; | |||
bool use_enzyme = false; | |||
bool isVectorValued = false; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
warning: member variable 'isVectorValued' has protected visibility [cppcoreguidelines-non-private-member-variables-in-classes]
bool isVectorValued = false;
^
Partially addresses #721.
22c027e
to
21939fb
Compare
clang-tidy review says "All clean, LGTM! 👍" |
Partially addresses #721.