[PATCH] D116851: [MemoryBuiltins] Add field for alignment argument

[Bryce Wilson via Phabricator via llvm-commits]

Bryce-MW created this revision.
Herald added subscribers: ormris, okura, jdoerfert, kuter, hiraditya.
Bryce-MW requested review of this revision.
Herald added a reviewer: jdoerfert.
Herald added a reviewer: jdoerfert.
Herald added a reviewer: sstefan1.
Herald added subscribers: llvm-commits, sstefan1.
Herald added a reviewer: baziotis.
Herald added a project: LLVM.

There are a few places where the alignment argument for AlignedAllocLike functions was previously hardcoded. This goes along with removing hardcoded arguments for the allocation size. It also allows alligned operator new to be given an alignment value in InstCombinerImpl::annotateAnyAllocSite and makes Heap To Stack add the proper alignment for aligned new. I'm sure this isn't the best way to do this but I am new to the LLVM codebase so I appreciate any suggestions on how to improve.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D116851

Files:
  llvm/include/llvm/Analysis/MemoryBuiltins.h
  llvm/lib/Analysis/MemoryBuiltins.cpp
  llvm/lib/Transforms/IPO/AttributorAttributes.cpp
  llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D116851.398299.patch
Type: text/x-patch
Size: 12698 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220108/ba47c296/attachment.bin>