This is an archive of the discontinued LLVM Phabricator instance.

Precommit FNeg InstCombine tests
ClosedPublic

Authored by cameron.mcinally on May 8 2019, 8:40 AM.

Details

Summary

Here are some new unary FNeg tests that copy the binary pseudo-FNeg tests.

Submitted for review to make sure everyone agrees on the weaved format with the existing fsub tests. Also, is everyone okay with simply replacing fneg->fsub in the old test names? Or should it be something more elaborate?

Diff Detail

Repository
rL LLVM

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptMay 8 2019, 8:40 AM
spatel accepted this revision.May 8 2019, 11:18 AM

LGTM - our 'test' dir standards are pretty low, so having any kind of meaningful names and/or organization is an improvement.

This revision is now accepted and ready to land.May 8 2019, 11:18 AM
This revision was automatically updated to reflect the committed changes.