[PATCH] D52834: [ARM] Account for implicit IT when calculating inline asm size

Peter Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 4 06:01:15 PDT 2018


peter.smith updated this revision to Diff 168278.
peter.smith added a comment.

Thanks for the comments. I've made an attempt at a diff where the MaxInstLength is increased to 6. This moves all the significant code to getInstSizeInBytes(). I've applied the change to all Targets (Elf, Macho, Windows ...) as it is possible to use implicit-IT on all of them.


https://reviews.llvm.org/D52834

Files:
  lib/Target/ARM/ARMBaseInstrInfo.cpp
  lib/Target/ARM/MCTargetDesc/ARMMCAsmInfo.cpp
  test/CodeGen/ARM/cbz-implicit-it-range.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D52834.168278.patch
Type: text/x-patch
Size: 4025 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181004/e7897ef7/attachment.bin>


More information about the llvm-commits mailing list