This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Tidy up some simple expressions for clarity NFC
ClosedPublic

Authored by dstuttard on Apr 30 2021, 2:48 AM.

Details

Summary

Slight refactor for clarity.

Change-Id: Ib25e7f4582c67a7c57f066cfd5382c1405d7d4c5

Diff Detail

Event Timeline

dstuttard created this revision.Apr 30 2021, 2:48 AM
dstuttard requested review of this revision.Apr 30 2021, 2:48 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 30 2021, 2:48 AM
foad accepted this revision.Apr 30 2021, 2:59 AM

Looks obviously fine.

llvm/lib/Target/AMDGPU/SIInstrInfo.cpp
29

Do you need this? I thought it was already included by default (not sure exactly how, though).

This revision is now accepted and ready to land.Apr 30 2021, 2:59 AM
dstuttard updated this revision to Diff 341826.Apr 30 2021, 3:14 AM

Extra include not required

dstuttard marked an inline comment as done.Apr 30 2021, 3:14 AM
This revision was landed with ongoing or failed builds.Apr 30 2021, 3:15 AM
This revision was automatically updated to reflect the committed changes.