https://bugs.llvm.org/show_bug.cgi?id=51245

Sanjay Patel <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #6 from Sanjay Patel <[email protected]> ---
(In reply to Simon Pilgrim from comment #5)
> Please can you ensure we have test coverage for more than 2 fcmps (with
> TODOs)? 
> 
> Then we can close this ticket and address the remaining TODOs (3+ fcmps,
> mixed float/double fcmp etc.) in fcmp-logic.ll one by one.
> 
> e.g.
> 
> bool f32cmp3(float x, float y, float z, float w) {
>     return ((x > 0) || (y > 0)) != (z < w);
> }

Added regression test corresponding to that example:
https://reviews.llvm.org/rG97948620b1ac

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
llvm-bugs mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs

Reply via email to