[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