[PATCH] D102197: [OpenMP] Enable HeapToStack conversion in OpenMPOpt for new RTL globalization calls

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert accepted this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.

LGTM, two nits



================
Comment at: llvm/lib/Transforms/IPO/OpenMPOpt.cpp:1651
+    };
+    GlobalizationRFI.foreachUse(SCC, CreateAA);
+
----------------
Run heap two stack for all functions if we have at least one kernel.


================
Comment at: llvm/test/Transforms/OpenMP/remove_globalization.ll:1
+; RUN: opt -S -passes=openmp-opt  < %s | FileCheck %s
+target datalayout = "e-i64:64-i128:128-v16:16-v32:32-n16:32:64"
----------------
use the update script please.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102197