[PATCH] D115888: [Attributor][Fix] Add default alignment to HeapToStack

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 16 10:28:03 PST 2021


lebedev.ri requested changes to this revision.
lebedev.ri added a comment.
This revision now requires changes to proceed.

In D115888#3198155 <https://reviews.llvm.org/D115888#3198155>, @jhuber6 wrote:

> In D115888#3198139 <https://reviews.llvm.org/D115888#3198139>, @lebedev.ri wrote:
>
>> Shouldn't/can't you query the alignment for that pointer?
>> Hardcoding anything like this is a sign of a problem.
>
> I haven't found an interface in LLVM to query default alignment information from malloc.

I'm talking about attributor's `AAAlign` attribute here.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D115888/new/

https://reviews.llvm.org/D115888



More information about the llvm-commits mailing list