[PATCH] [SLPVectorizer] Reorder operands of shufflevector if it can result in a vectorized code.
kv.bhat at samsung.com
Wed Jan 14 23:35:06 PST 2015
Thanks for the reveiw. Addressed all review comments.
I'm ok with the current way we are handling reorderInputsAccordingToOpcode. If we were to move the load matching before checking AllSameOpcodeRight we would need to maintain a flag so that we do not reorder operands that were swapped during consecutive load matching.(This would end up similar to the initial version of the patch that was uploaded with needsReordering flag). I do not see much difference in the 2 approaches. I hope we can keep this as it is?
Please let me know if i can go ahead and commit this into mainline or if you have any other comments.
Thanks a lot.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 23948 bytes
Desc: not available
More information about the llvm-commits