Please use GitHub pull requests for new patches. Avoid migrating existing patches. Phabricator shutdown timeline

Differential D48180

updating isNegatibleForFree and GetNegatedExpression with fmf for fadd
ClosedPublic
Actions

Authored by mcberg2017 on Jun 14 2018, 10:03 AM.

Tags

None

Subscribers

Details

Reviewers

spatel
wristow
arsenm
hfinkel

Commits

rG4663ceb63f89: updating isNegatibleForFree and GetNegatedExpression with fmf for fadd
rL334753: updating isNegatibleForFree and GetNegatedExpression with fmf for fadd

Summary

A FMF constraint is added to FADD with unsafe still available as the fallback

Diff Detail

Event Timeline

mcberg2017 created this revision.Jun 14 2018, 10:03 AM

Herald added a subscriber: wdng. · View Herald TranscriptJun 14 2018, 10:03 AM

LGTM.

spatel accepted this revision.Jun 14 2018, 10:05 AM

This revision is now accepted and ready to land.Jun 14 2018, 10:05 AM

Closed by commit rL334753: updating isNegatibleForFree and GetNegatedExpression with fmf for fadd (authored by mcberg2017). · Explain WhyJun 14 2018, 11:53 AM

This revision was automatically updated to reflect the committed changes.

Herald added a subscriber: llvm-commits. · View Herald TranscriptJun 14 2018, 11:53 AM

Revision Contents
Changeset List

Path

Size

lib/

CodeGen/

SelectionDAG/

DAGCombiner.cpp

7 lines

test/

CodeGen/

X86/

11 lines

Diff 151373

lib/CodeGen/SelectionDAG/DAGCombiner.cpp

Loading...

test/CodeGen/X86/fp-fold.ll

Loading...