[PATCH] D14148: [GlobalOpt] Demote globals to locals more aggressively

[Mehdi AMINI via llvm-commits]

joker.eph added inline comments.

================
Comment at: lib/Transforms/IPO/GlobalOpt.cpp:1809
@@ -1741,1 +1808,3 @@
+      GS.AccessingFunction->doesNotRecurse() &&
+      isPointerValueDeadOnEntryToFunction(GS.AccessingFunction, GV) ) {
     DEBUG(dbgs() << "LOCALIZING GLOBAL: " << *GV << "\n");
----------------
If the function does not recurse and is the only user of the global, why do we need `isPointerValueDeadOnEntryToFunction`?



Repository:
  rL LLVM

http://reviews.llvm.org/D14148