[PATCH] D122558: [2/3] TLS loads optimization (hoist)

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

The problem isn't really the amount of code needed to support the option... it's that a formally documented option needs to have clear semantics, and we need to support it forever.  We really don't like options that say "turn off optimization X that has no user-visible semantics".  I mean, even if you pass -mno-tls-load-hoist, we might end up hoisting anyway, depending on what you're doing with the address...


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

https://reviews.llvm.org/D122558