[PATCH] D106872: [InstCombine] Remove nnan requirement for transformation to fabs from select

Tue Aug 3 06:44:56 PDT 2021

Krishnakariya added a comment.

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

> In D106872#2922101 <https://reviews.llvm.org/D106872#2922101>, @Krishnakariya wrote:
>
>> I committed this patch by mistake. I am extremely sorry about this. 
>> I've reverted the commit.
>
> FWIW reading through the disscussion i would think this has been basically accepted, if not stamped yet for formal reasons of wanting the tests to be precommitted.

I have pre-committed the tests and updated the diff. Commit link: https://reviews.llvm.org/rG56e7b6c3924d7ba8db70c38235a77ed8208795eb .

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106872