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
[Intel MKL]code cleanup #33703
[Intel MKL]code cleanup #33703
Conversation
@gbaned @adisakshya This PR has pending on ready to pull for 5 days, can you help to check this? thanks. |
@@ -637,6 +638,7 @@ bool FindContractionWithBiasAndAddActivation( | |||
|
|||
// Root of the pattern must be an activation node. | |||
const auto* node_def = node_view->node(); | |||
if (node_def == nullptr) return 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.
Did this happen in practice? It's probably a bug somewhere if node def in null.
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.
@ezhulenev No, this just pass some tools check.
PiperOrigin-RevId: 278923777 Change-Id: I9b8db719e535efe25ed2ff67495a19704c699b23
No description provided.