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

[Xiang Zhang via Phabricator via llvm-commits]

xiangzhangllvm 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();
----------------
LuoYuanke wrote:
> 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?
Yes, no problem


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

https://reviews.llvm.org/D120000