cameron.mcinally added a comment.

In D71767#2206947 <https://reviews.llvm.org/D71767#2206947>, @paulwalker-arm 
wrote:

> @cameron.mcinally this is the patch I mentioned the other day, which contains 
> the nodes where once I've written suitable tests I'll push separate patches 
> for.

Thanks, Paul. You mentioned that you would be focusing on another project for a 
few weeks. Would it help if I attempted to cherry-pick some of this Diff into 
individual patches (with new tests) for you? Or would I be stepping on your 
toes too much?

> Anything else is fair game.  This patch implements VSELECT but that was just 
> to investigate what we talked about during the previous sync call so I'll 
> ignore it if you're planning to push on with your original work?

I can carry on with VSELECT. I think I have enough intuition built around it 
now. I haven't yet looked into the state of the unpack[lo|hi] instructions, but 
will do so today.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71767/new/

https://reviews.llvm.org/D71767

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to