This is an archive of the discontinued LLVM Phabricator instance.

[flang][driver] Add missing parentheses in an assert
ClosedPublic

Authored by awarzynski on May 5 2022, 11:02 AM.

Details

Summary

The assert in https://reviews.llvm.org/D124665 was missing parentheses,
which triggered a warning in GCC (verified with GCC 11). As -Werror is
on by default in FLang, that triggered build errors, see e.g. [1].

The fix is rather straightforward, so I am sending this without a
review.

[1] https://lab.llvm.org/buildbot/#/builders/160/builds/7016

Diff Detail

Event Timeline

awarzynski created this revision.May 5 2022, 11:02 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptMay 5 2022, 11:02 AM
awarzynski requested review of this revision.May 5 2022, 11:02 AM
This revision was not accepted when it landed; it landed in state Needs Review.May 5 2022, 11:03 AM
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.