[PATCH] D77319: [DAGCombine] Remove the getNegatibleCost to avoid the out of sync with getNegatedExpression

[Qing Shan Zhang via Phabricator via llvm-commits]

steven.zhang updated this revision to Diff 258239.
steven.zhang added a comment.

Split this patch into three parts for easy to review:

1. D78291 <https://reviews.llvm.org/D78291> NFC change by adding new helper functions and the changes in DAGCombine.
2. D78347 <https://reviews.llvm.org/D78347> Addressing the reason of the change in PowerPC tests.
3. D77319 <https://reviews.llvm.org/D77319>(this patch) Remove the getNegatibleCost and merge it with getNegatedExpression to fix the out of sync issue.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77319

Files:
  llvm/include/llvm/CodeGen/TargetLowering.h
  llvm/lib/CodeGen/SelectionDAG/TargetLowering.cpp
  llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp
  llvm/lib/Target/AMDGPU/AMDGPUISelLowering.h
  llvm/lib/Target/X86/X86ISelLowering.cpp
  llvm/lib/Target/X86/X86ISelLowering.h
  llvm/test/CodeGen/X86/neg_fp.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D77319.258239.patch
Type: text/x-patch
Size: 27192 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200417/6bd23e66/attachment.bin>