This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU][NFC] Remove redundant condition
ClosedPublic

Authored by baloghadamsoftware on Jun 25 2020, 7:49 AM.

Details

Summary

Condition LiteralCount is checked both in an outer and in an inner if statement in SIInstrInfo::verifyInstruction(). This patch removes the redundant inner check.

The issue was found using clang-tidy check under review misc-redundant-condition. See https://reviews.llvm.org/D81272.

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptJun 25 2020, 7:49 AM
arsenm accepted this revision.Jun 25 2020, 9:49 AM
This revision is now accepted and ready to land.Jun 25 2020, 9:49 AM
This revision was automatically updated to reflect the committed changes.