[PATCH] D83859: [SVE] Don't use LocalStackAllocation for SVE objects

[David Sherwood via Phabricator via llvm-commits]

david-arm created this revision.
david-arm added reviewers: sdesmalen, efriedma, paulwalker-arm, kmclaughlin.
Herald added subscribers: llvm-commits, psnobl, hiraditya, kristof.beyls, tschuett.
Herald added a reviewer: rengolin.
Herald added a project: LLVM.

I have introduced a new TargetFrameLowering query function:

  bundleStackIdIntoLocalArea

that queries whether or not objects of a given stack id can be
bundled into the local area. The default behaviour is to always
bundle regardless of the stack id, however for AArch64 this is
overriden where we only bundle for non-SVE stack ids. It's not
a trivial amount of effort to support putting SVE objects in the
pre-allocated local frame block and we haven't done any cost
analysis so far to prove whether this is profitable or not.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D83859

Files:
  llvm/include/llvm/CodeGen/TargetFrameLowering.h
  llvm/lib/CodeGen/LocalStackSlotAllocation.cpp
  llvm/lib/Target/AArch64/AArch64FrameLowering.h
  llvm/test/CodeGen/AArch64/sve-localstackalloc.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D83859.278099.patch
Type: text/x-patch
Size: 5166 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200715/db5dd6a9/attachment.bin>