User Details
- User Since
- May 7 2019, 7:55 AM (100 w, 6 d)
Thu, Apr 1
Merged more recent changes on main.
Revert diff 334860
Remove unused header
Remove unused header include
Fri, Mar 26
Thanks Hussain for this patch!
For now I have a couple of minor comments, I will take a deeper look later.
Also, I don't quite understand yet how VP handles (or plans to handle) specifying alignment parameter. Does it need VPBuilder (not to be confused with VPlan VPBuilder!) for that? Would it make sense to add the Alignment parameter to VP load/store/scatter/gather intrinsics (much like their masked counterparts) for now, unless there is already a way to specify that?
Feb 1 2021
Jan 29 2021
Jan 27 2021
FWIW, we have a similar intrinsic in our downstream compiler for RISC-V. We call it experimental.vector.slideleftfill and is same as flavour 2 of this (except the "offset" need not be an immediate) as suggested by @sdesmalen. We have a separate predicated version (experimental.vector.vp.slideleftfill) too which takes additional arguments for the explicit vector length of the two vectors.
Nov 5 2020
LGTM.
Other than the minor nit, LGTM.
Oct 29 2020
Overall, definitely a good starting point. I just have a couple of minor comments.
Oct 5 2020
Sep 30 2020
Aug 14 2020
Aug 13 2020
Thanks @fpetrogalli for this proposal! I agree with @rogfer01 and @rengolin in that this approach makes a lot of sense.
I have added a few comments inline. Overall looks good.