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

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

One minor concern; otherwise seems fine.



================
Comment at: llvm/lib/Transforms/Scalar/TLSVariableHoist.cpp:52
+        "tls-load-hoist=non-optimize: Generally load TLS before use(s)."),
+    cl::init("non-optimize"), cl::Hidden);
+
----------------
Why is this off by default?  Do you plan to turn it on by default in a followup?


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

https://reviews.llvm.org/D120000