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

[Xiang Zhang via Phabricator via llvm-commits]

xiangzhangllvm added a comment.

In D120000#3348559 <https://reviews.llvm.org/D120000#3348559>, @lebedev.ri wrote:

> What about profitability, is it always a win?

It remove duplicated TLS address call, In theory, it has big profitability to win. 
But similar with most optimization, I can't say it must win in any case.

>> Sorry, what you mean "opt-in" ?
>
> Why is it not enabled by default?

This is a general optimization which will affect all targets test, I'd like to enable it by another independent patch with mainly modify tests.


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

https://reviews.llvm.org/D120000