-
Notifications
You must be signed in to change notification settings - Fork 407
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
ThreadVectorRange with a range #1400
Comments
OK OK we will do it 👎 ... We didn't have that initially because it is better for the compiler and alignment etc. to have vectorized loops which go from 0 to N. But
So we might as well do it. |
Oh.... I see why you prefer not to have it. Thanks. I meant to use the Thank you again. |
Kyungjoo I totally get it, and I think we should just do it, but document that if possible use the compile time zero offset variant. |
I just thought I'd need this feature too, and I stumbled on this issue. Any rough idea if/when this could end up in master (or at least develop)? To be clear, I'm not trying to put pressure, I'm just trying to understand if I can wait for this, or if I should find a workaround. Thanks! |
Add overload to accept a range and add ctors to ThreadVectorRangeBoundariesStruct to allow for a starting index. Address issue kokkos#1400.
I found for
TeamThreadRange(member, ibeg, iend)
but I could not seeThreadVectorRange(member, ibeg, iend)
. Do we have a specifc reason not to have that ? If everything is okay, would it be possible to add a constructor with a range arguments (member, begin, end) ?The text was updated successfully, but these errors were encountered: