[PATCH] D32629: [StackMaps] Increase the size of the "location size" field

Sanjoy Das via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 27 17:54:10 PDT 2017


sanjoy created this revision.
Herald added subscribers: nemanjai, mcrosier.
Herald added a reviewer: javed.absar.

In some cases LLVM (especially the SLP vectorizer) will create vectors
that are 256 bytes (or larger).  Given that this is intentional[0] is
likely to get more common, this patch updates the StackMap binary
format to deal with the spill locations for said vectors.

This change also bumps the stack map version from 2 to 3.

[0]: https://reviews.llvm.org/D32533#738350


https://reviews.llvm.org/D32629

Files:
  docs/StackMaps.rst
  lib/CodeGen/StackMaps.cpp
  test/CodeGen/AArch64/arm64-anyregcc.ll
  test/CodeGen/AArch64/arm64-stackmap.ll
  test/CodeGen/AArch64/stackmap-liveness.ll
  test/CodeGen/PowerPC/ppc64-anyregcc.ll
  test/CodeGen/PowerPC/ppc64-stackmap.ll
  test/CodeGen/X86/anyregcc.ll
  test/CodeGen/X86/deopt-bundles.ll
  test/CodeGen/X86/deopt-intrinsic-cconv.ll
  test/CodeGen/X86/deopt-intrinsic.ll
  test/CodeGen/X86/patchpoint-invoke.ll
  test/CodeGen/X86/stackmap-fast-isel.ll
  test/CodeGen/X86/stackmap-large-constants.ll
  test/CodeGen/X86/stackmap-large-location-size.ll
  test/CodeGen/X86/stackmap-liveness.ll
  test/CodeGen/X86/stackmap.ll
  test/CodeGen/X86/statepoint-allocas.ll
  test/CodeGen/X86/statepoint-live-in.ll
  test/CodeGen/X86/statepoint-stackmap-format.ll
  test/CodeGen/X86/statepoint-vector.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D32629.97031.patch
Type: text/x-patch
Size: 95747 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170428/7143a81e/attachment.bin>


More information about the llvm-commits mailing list