[PATCH] D34485: [AMDGPU] Add FP_CLASS to the add/setcc combine

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec added a comment.

In https://reviews.llvm.org/D34485#787393, @rampitec wrote:

> In https://reviews.llvm.org/D34485#787380, @arsenm wrote:
>
> > LGTM. Should this also handle i1 and/or/xor?
>
>
> I'm not sure yet, I have to check the resulting code. Probably it should.


Looks like it is beneficial for anything producing i1 and which is an instruction. I.e. not for arguments, constants etc.


Repository:
  rL LLVM

https://reviews.llvm.org/D34485