[PATCH] D42999: [TLS] use emulated TLS if the target supports only this mode

Stephen Hines via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 13 09:17:32 PST 2018


srhines added inline comments.


================
Comment at: lib/Target/TargetMachine.cpp:195
+  return Triple.isAndroid() || Triple.isOSOpenBSD()
+      || Triple.isWindowsCygwinEnvironment();
+}
----------------
Can we move this into a function in Triple as well? I feel like keeping this as a property there makes it more likely to be handled for future users as well.


https://reviews.llvm.org/D42999





More information about the llvm-commits mailing list