[PATCH] D50629: AMDGPU: Fix getInstSizeInBytes

Nicolai Hähnle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 22 06:58:31 PDT 2018


nhaehnle updated this revision to Diff 161948.
nhaehnle added a comment.
Herald added a subscriber: jvesely.

Always do the sanity check in debug builds.

I was originally worried about how expensive this is, but it
doesn't show up in the running time of all the tests in
test/CodeGen/AMDGPU.


Repository:
  rL LLVM

https://reviews.llvm.org/D50629

Files:
  lib/Target/AMDGPU/AMDGPUMCInstLower.cpp
  lib/Target/AMDGPU/SIInstrInfo.cpp
  lib/Target/AMDGPU/VOP2Instructions.td
  lib/Target/AMDGPU/VOP3Instructions.td
  test/CodeGen/AMDGPU/llvm.amdgcn.writelane.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D50629.161948.patch
Type: text/x-patch
Size: 7562 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180822/d90a8536/attachment.bin>


More information about the llvm-commits mailing list