This is an archive of the discontinued LLVM Phabricator instance.

Have lldb_assert accept bool expressions
ClosedPublic

Authored by labath on May 7 2015, 8:15 AM.

Details

Summary

This changes lldb_assert to accept bool expressions as the parameter, this is because some
objects (such as std::shared_ptr) are convertible to bool, but are not convertible to int, which
leads to surprising errors.

Diff Detail

Repository
rL LLVM

Event Timeline

labath updated this revision to Diff 25191.May 7 2015, 8:15 AM
labath retitled this revision from to Have lldb_assert accept bool expressions.
labath updated this object.
labath edited the test plan for this revision. (Show Details)
labath added reviewers: granata.enrico, zturner.
labath added a subscriber: Unknown Object (MLST).
granata.enrico edited edge metadata.May 7 2015, 9:03 AM

LGTM

  • Enrico
This revision was automatically updated to reflect the committed changes.