[PATCH] D123830: [InstCombine] Optimize folding of fneg-of-fabs

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

This doesn't solve the root problem. If we start with the existing code in the test that is changing, it will not become fneg(fabs) like we want?
https://alive2.llvm.org/ce/z/e_uhUQ

We should extend the code under InstCombinerImpl::visitSelectInst() to match "nabs" patterns. Search for: "// Canonicalize select with fcmp to fabs()."


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123830