[PATCH] D125601: [DAGCombiner][AArch64] Reorder the bitcast of scalable vector

[Paul Walker via Phabricator via llvm-commits]

paulwalker-arm added a comment.

Hi @Allen, sorry I've not had chance to review your patch yet.  I'll try to take a proper look tomorrow but in the interim my main concern is that this optimisation specifically relates to NEON sized vectors where we can do a better job of the extracts.  I'm not sure we want to do this transform for all fixed length vector types though.  Part of me thinks we're missing a different, more specific, transformation somewhere.  However, I cannot be sure until I've had chance to investigate a little more.


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

https://reviews.llvm.org/D125601