[PATCH] D109049: [SLP] Support llvm.isnan in vectorizer

[Serge Pavlov via Phabricator via llvm-commits]

sepavloff added a comment.

In D109049#2976514 <https://reviews.llvm.org/D109049#2976514>, @lebedev.ri wrote:

> In D109049#2976512 <https://reviews.llvm.org/D109049#2976512>, @sepavloff wrote:
>
>> In D109049#2976449 <https://reviews.llvm.org/D109049#2976449>, @xbolva00 wrote:
>>
>>> But there are concerns in https://reviews.llvm.org/D104854 whether the base change should not be reverted at all…
>>
>> There is a discussion: https://lists.llvm.org/pipermail/llvm-dev/2021-August/152257.html, devoted to this topic. There are no additional feedback during the last week, so I would think the problem is closed.
>
> Au contraire.
> It looks like goalpost shifting to me.
> What is going on the is basic design disscussion, with no clear favorite,
> even though a design has already been "selected" and committed.
> I would like to once again urge you to restart this process.

What exactly do you want me to restart? There is already a discussion on this topic, it could be continued. You could express your concerns there to warm the discussion.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109049