-
Notifications
You must be signed in to change notification settings - Fork 76
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
adding multiply primitive #38
Conversation
std::vector<primitive> operands_; | ||
}; | ||
}}} | ||
|
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.
Something went wrong with the indentation here.
); | ||
} | ||
}}} | ||
|
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.
Same here, indentation gone wrong
May I suggest you reformat all source files using clang-format (there is a .clang-format configuration file in the root of this repository). |
667709d
to
0a8ee2f
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.
LGTM, thanks a lot!
This adds multiply primitive. It works for vector of operands when:
issue: when tested with matrices larger than 100 by 100 got this:
Stack overflow in coroutine at address 0x00007f1286f2fbb8. Configure the hpx runtime to allocate a larger coroutine stack size. But the logic works and tests pass for smaller matrices.