[PATCH] D106043: [AMDGPU] Use isMetaInstruction for instruction size

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 15 02:52:58 PDT 2021


foad accepted this revision.
foad added a comment.
This revision is now accepted and ready to land.

Looks good.

Typo in summary "explicitly".

Could put the isMetaInstruction check inside the default case of the switch, to save a nanosecond in the common case.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D106043/new/

https://reviews.llvm.org/D106043



More information about the llvm-commits mailing list