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

Thorsten via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 16 12:04:45 PST 2021


tschuett added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/AttributorAttributes.cpp:5931
 
-      Align Alignment(1);
+      Align Alignment(16);
       if (AI.Kind == AllocationInfo::AllocationKind::ALIGNED_ALLOC) {
----------------
Would a comment help to explain what the hard-coded 16 means?


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