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

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/TLSVariableControl.cpp:1
+//===- TLSVariableControl.cpp - Remove Redundant TLS Loads --------===//
+//
----------------
TLSVariableControl doesn't seem like a good name for the pass if the description doesn't use the words "variable" or "control". Does "control" have some meaning with TLS?


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

https://reviews.llvm.org/D120000