This is an archive of the discontinued LLVM Phabricator instance.

[AsmWriter] Eliminate warning. NFC
ClosedPublic

Authored by skatkov on Apr 20 2017, 10:14 PM.

Details

Summary

This patch eliminates the following warning

lib/IR/AsmWriter.cpp:1128:57: warning: suggest parentheses around ‘&&’ within ‘||’ [-Wparentheses]

(StrVal[1] >= '0' && StrVal[1] <= '9')) &&

Diff Detail

Repository
rL LLVM

Event Timeline

skatkov created this revision.Apr 20 2017, 10:14 PM
davide accepted this revision.Apr 20 2017, 11:03 PM
davide added a subscriber: davide.

LG. No need to submit review for fixing trival warnings like this.

This revision is now accepted and ready to land.Apr 20 2017, 11:03 PM

Ok, Thank you!

The reason I posted this for a review is that when i've got a committer access, e-mail said: "commit after approval access granted".

I thought that I should post all patches for the review.

This revision was automatically updated to reflect the committed changes.