[PATCH] R600/SI: Use source modifiers for f64 fabs / fneg

Matt Arsenault Matthew.Arsenault at amd.com
Fri Aug 1 19:33:56 PDT 2014


f32 has a comment and patterns about the behavior with 0 and negate modifier which I'm unsure about. I think the source modifier should work if the source is 0, since the negate modifier is supposed to be applied before abs. Does this pattern need to be skipped for f64 too?

http://reviews.llvm.org/D4767

Files:
  lib/Target/R600/AMDGPUISelLowering.cpp
  lib/Target/R600/SIISelLowering.cpp
  lib/Target/R600/SIInstructions.td
  test/CodeGen/R600/fabs.f64.ll
  test/CodeGen/R600/fneg-fabs.f64.ll
  test/CodeGen/R600/fneg.f64.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D4767.12138.patch
Type: text/x-patch
Size: 13743 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140802/08c5cb54/attachment.bin>


More information about the llvm-commits mailing list