Conversation
it was possible for a ConstAccessorSimple to be dropped before its corresponding ConstAccessorPatchSimple
|
I think you found the reason why this was so kinda complicated in the first place, with the Accessor replicating While I introduced the accessor stuff for a reason (likely actually because of CUDA), I'd actually rather get rid of this complexity again in some future vision of CPU/GPU compatability more akin to how gtensor does stuff. |
germasch
left a comment
There was a problem hiding this comment.
I guess there isn't actually a "reject" option for PR reviews, but well, I think that's my choice.
This makes it safe to do
mprts.accessor()[p]without having to worry about the result ofmprts.accessor()being dropped.This was happening in #354, in particular.