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

Thu Dec 16 11:39:04 PST 2021

jhuber6 added a comment.

In D115888#3198178 <https://reviews.llvm.org/D115888#3198178>, @lebedev.ri wrote:

> Then i guess you need to basically introduce an interface to do what https://en.cppreference.com/w/cpp/types/max_align_t does, but based on a datalayout.

Seems reasonable. I know we can query this information from clang, e.g. https://clang.llvm.org/doxygen/classclang_1_1TargetInfo.html#a01403a5106161d4d3cd0c50c43150f89, but I don't think there is an existing string in the data layout to encode this. Will I be adding a new format this? That would be a reasonably large change so I just want to make sure I'm on the right page.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115888