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
Fixing component handling for lcos #2856
Conversation
This patch fixes a UB where the LCOs using (simple) components where cast to wrong types.
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.
Otherwise, LGTM, thanks!
protected: | ||
hpx::performance_counters::counter_info info_; | ||
util::atomic_count invocation_count_; | ||
class base_performance_counter |
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.
What happened 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.
This diff are reformating changes introduced by clang-format...
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.
We agreed not to reformat whole files without dire need...
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.
I have no problem reverting the formatting changes again. I'm not exactly sure what happened here. I use git clang-format
which should only affect the direct changeset (in that case, a forward declaration was removed). The resulting diff is pretty awful...
- Moving implementation of a simple component to component(_base) - Keeping using simple_component = component and friends for backwards compatbility.
Conflicts: hpx/runtime/components/server/simple_component_base.hpp
This patch fixes a UB where the LCOs using (simple) components where cast to
wrong types.