* Re: [PATCH, rs6000] vec_mule and vec_mulo builtin fix [not found] <1500591361.19855.44.camel@us.ibm.com> @ 2017-07-21 20:43 ` Segher Boessenkool 2017-07-21 21:15 ` Segher Boessenkool 1 sibling, 0 replies; 2+ messages in thread From: Segher Boessenkool @ 2017-07-21 20:43 UTC (permalink / raw) To: Carl Love; +Cc: gcc-patches, David Edelsohn, Bill Schmidt On Thu, Jul 20, 2017 at 03:56:01PM -0700, Carl Love wrote: > The following patch is a reworked patch to fix the bugs in the vec_mule > and vec_mulo patch that had to be reverted. The reverted fix, added the > correct mule and mulo instructions with vectorization. The > vectorization support resulted in vectorization in several testcases to > fail. This patch adds the correct instructions for the vec_mule and > vec_mulo builtins without vectorization. The goal is add the > vectorization support again in a future patch. The patch looks fine to me, but so did the previous version... What has changed? It's not clear to me. Segher ^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH, rs6000] vec_mule and vec_mulo builtin fix [not found] <1500591361.19855.44.camel@us.ibm.com> 2017-07-21 20:43 ` [PATCH, rs6000] vec_mule and vec_mulo builtin fix Segher Boessenkool @ 2017-07-21 21:15 ` Segher Boessenkool 1 sibling, 0 replies; 2+ messages in thread From: Segher Boessenkool @ 2017-07-21 21:15 UTC (permalink / raw) To: Carl Love; +Cc: gcc-patches, David Edelsohn, Bill Schmidt On Thu, Jul 20, 2017 at 03:56:01PM -0700, Carl Love wrote: > The following patch is a reworked patch to fix the bugs in the vec_mule > and vec_mulo patch that had to be reverted. The reverted fix, added the > correct mule and mulo instructions with vectorization. The > vectorization support resulted in vectorization in several testcases to > fail. This patch adds the correct instructions for the vec_mule and > vec_mulo builtins without vectorization. The goal is add the > vectorization support again in a future patch. Ah, as you explained offline, you removed vec_widen_umult_even_v4si and friends. Okay :-) The patch is okay for trunk then. Thanks! Segher ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-07-21 21:15 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <1500591361.19855.44.camel@us.ibm.com>
2017-07-21 20:43 ` [PATCH, rs6000] vec_mule and vec_mulo builtin fix Segher Boessenkool
2017-07-21 21:15 ` Segher Boessenkool
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).