[PATCH] D47345: [InstructionCombining] Replace small heap allocations with local variables

[Dávid Bolvanský via Phabricator via llvm-commits]

xbolva00 added inline comments.


================
Comment at: lib/Transforms/InstCombine/InstructionCombining.cpp:2276
 
+bool InstCombiner::eliminateSmallAllocations(Value *Op, CallInst &FI) {
+  Function *F = FI.getFunction();
----------------
xbolva00 wrote:
> Or maybe we can turn larger allocations into global variables also? + allocations with unknown (not constant) size?
Ah, possible data race on that global var..


https://reviews.llvm.org/D47345