[PATCH] D53103: Support for the mno-tls-direct-seg-refs flag

[Ruslan Nikolaev via Phabricator via llvm-commits]

nruslan marked 3 inline comments as done.
nruslan added inline comments.


================
Comment at: lib/Target/X86/X86ISelDAGToDAG.cpp:987
     if (C->getSExtValue() == 0 && AM.Segment.getNode() == nullptr &&
+        !IndirectTlsSegRefs &&
         (Subtarget->isTargetGlibc() || Subtarget->isTargetAndroid() ||
----------------
hans wrote:
> Looks very simple :-) I'm not really familiar with this, but I assume you've tested that it does the right thing.
Yeah, it seems to pass tests. I also added one more test.


================
Comment at: test/CodeGen/X86/tls.ll:457
+
+define i32 @f17() #0 {
+; X86_LINUX-LABEL: f17:
----------------
hans wrote:
> I know this file seems to go for brevity of function names, but I'd still suggest calling it something more descriptive, or adding a comment so it's more obvious to the reader what's going on.
added a comment there


Repository:
  rL LLVM

https://reviews.llvm.org/D53103