[PATCH] D65408: [Attributor] Heap-To-Stack Conversion

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 29 15:37:14 PDT 2019


jdoerfert added a comment.

Top-level functions declarations should have attributes to make them as restricted as needed, e.g. `sync_func` should probably be `nofree` or renamed to `sync_free_func`.



================
Comment at: llvm/test/Transforms/FunctionAttrs/heap_to_stack.ll:48
+  ret void
+}
+
----------------
What is the difference between Test 2 and 3?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65408





More information about the llvm-commits mailing list