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

LuoYuanke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 1 03:51:45 PDT 2022


LuoYuanke added a comment.

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

> I'm not really sure that attribute should be there, especially after the optimization is enabled by default and cl::opt is inverted.

I think we can leave end user a chance to enable it or not. Maybe for debug purpose.


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

https://reviews.llvm.org/D122890



More information about the llvm-commits mailing list