[PATCH] D106802: Adding flags for disabling the following optimizations: Deglobalization SPMDization State machine rewrites Folding

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/AttributorAttributes.cpp:71
+    cl::desc("Disable OpenMP optimizations involving deglobalization 2 of 2."),
+    cl::Hidden, cl::init(false));
+
----------------
I recommend to not seed AAHeapToStack / AAHeapToShared in OpenMP-Opt if the flag is set.
If you want to keep the current model use a external storage location that you can declare as extern in the other TU:
```
  static cl::opt<unsigned, true> MaxInitializationChainLengthX(
    ┊ "attributor-max-initialization-chain-length", cl::Hidden,
    ┊ cl::desc(
    ┊ ┊ ┊ "Maximal number of chained initializations (to avoid stack overflows)"),
    ┊ cl::location(MaxInitializationChainLength), cl::init(1024));
  unsigned llvm::MaxInitializationChainLength;
```


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106802