This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Fix assert message operator
ClosedPublic

Authored by mgrang on Apr 14 2018, 8:53 PM.

Details

Summary

Specifying assert message with an || operator makes the compiler interpret it
as a bool. Changed it to &&.

Diff Detail

Repository
rL LLVM

Event Timeline

mgrang created this revision.Apr 14 2018, 8:53 PM
asb accepted this revision.Apr 16 2018, 6:44 AM

Thanks, that's definitely an error. Please do feel totally free to commit straight-forward trivial fixes like this without the need for pre-commit review.

This revision is now accepted and ready to land.Apr 16 2018, 6:44 AM
This revision was automatically updated to reflect the committed changes.