Jonas Paulsson via llvm-dev
2017-Apr-10 14:06 UTC
[llvm-dev] Handling of shuffle instructions reverted.
Hi Regarding: https://reviews.llvm.org/rL180834 I tried my old patch for InstCombiner for shuffles, which was a handling for https://bugs.llvm.org//show_bug.cgi?id=30630. When I applied it a month ago, it produced nearly no change. But now I do see some files that needs this optimization, and I suspect it has to do with this revert. I wonder what the status is on getting this reverted patch recommitted? Should I post my patch for InstCombiner on Phabricator? /Jonas -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20170410/42db1925/attachment.html>
Friedman, Eli via llvm-dev
2017-Apr-10 17:20 UTC
[llvm-dev] Handling of shuffle instructions reverted.
On 4/10/2017 7:06 AM, Jonas Paulsson via llvm-dev wrote:> > Hi > > Regarding: https://reviews.llvm.org/rL180834 > > I tried my old patch for InstCombiner for shuffles, which was a > handling for https://bugs.llvm.org//show_bug.cgi?id=30630. When I > applied it a month ago, it produced nearly no change. But now I do see > some files that needs this optimization, and I suspect it has to do > with this revert. > > I wonder what the status is on getting this reverted patch > recommitted? Should I post my patch for InstCombiner on Phabricator? >See https://reviews.llvm.org/D31509. -Eli -- Employee of Qualcomm Innovation Center, Inc. Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20170410/c172d6c0/attachment.html>