This is an archive of the discontinued LLVM Phabricator instance.

Quiet Coverity
ClosedPublic

Authored by jhibbits on Feb 11 2015, 6:25 PM.

Details

Summary

Coverity warns that unsigned >= 0 is always true, and k_first_gpr_powerpc happens to be 0. Quiet Coverity by changing that comparison instead to a static_assert(), in case things change in the future.

Diff Detail

Repository
rL LLVM

Event Timeline

jhibbits updated this revision to Diff 19800.Feb 11 2015, 6:25 PM
jhibbits retitled this revision from to Quiet Coverity.
jhibbits updated this object.
jhibbits edited the test plan for this revision. (Show Details)
jhibbits added a reviewer: emaste.
jhibbits set the repository for this revision to rL LLVM.
jhibbits added a subscriber: Unknown Object (MLST).
emaste accepted this revision.Feb 11 2015, 6:51 PM
emaste edited edge metadata.
This revision is now accepted and ready to land.Feb 11 2015, 6:51 PM
This revision was automatically updated to reflect the committed changes.