This is an archive of the discontinued LLVM Phabricator instance.

DAG: Fix naming crime
ClosedPublic

Authored by arsenm on Aug 15 2017, 3:42 PM.

Details

Reviewers
bogner
arsenm
Summary

Because isOperationCustom was only checking for custom
lowering on illegal types, this was behaving inconsistently
with the other isOperation* functions, so that
isOperationLegalOrCustom != (isOperationLegal || isOperationCustom)

Luckily this is only used in one place which already checks the
type legality on its own.

Diff Detail

Event Timeline

arsenm created this revision.Aug 15 2017, 3:42 PM
arsenm accepted this revision.Aug 24 2017, 6:27 PM

r311743

This revision is now accepted and ready to land.Aug 24 2017, 6:27 PM
arsenm closed this revision.Aug 24 2017, 6:27 PM