[PATCH] D71993: [Transforms][GlobalSRA] huge array causes long compilation time and huge memory usage.

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/GlobalOpt.cpp:512-513
     unsigned NumElements = STy->getNumElements();
-    if (NumElements > 16 && GV->hasNUsesOrMore(16))
+    if (NumElements > 16)
       return nullptr; // It's not worth it.
     NewGlobals.reserve(NumElements);
----------------
It looks like this makes bailout more strict, therefore it likely regresses something else?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71993