[PATCH] D120000: [1/2] TLS loads opimization (hoist)

[LuoYuanke via Phabricator via llvm-commits]

LuoYuanke added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/TLSVariableHoist.cpp:186
+  BasicBlock &Entry = Fn.getEntryBlock();
+  BasicBlock::iterator Iter = Entry.getFirstInsertionPt();
+  Type *Ty = GV->getType();
----------------
xiangzhangllvm wrote:
> craig.topper wrote:
> > xiangzhangllvm wrote:
> > > craig.topper wrote:
> > > > Is this before the allocas?
> > > Sorry, don't much understand,
> > > What the problem if it before allocas ? 
> > > This is in IR level and the Global Value do not need "allocas" 
> > The alloca instructions for the function's local variables are the first instructons in the entry basic block. Not if we should be putting the bitcast before them.
> OK, Let me re-place the bitcast position.
Would you add a test case with alloca for checking?


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

https://reviews.llvm.org/D120000