[PATCH] D38392: Disallow sinking of unordered atomic loads into loops

Dehao Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 29 16:36:32 PDT 2017


danielcdh added a comment.

I see your point now. My concern is performance: if we allow hoisting of atomic load, but not allow sinking it, we may end up with bad performance as we may have too much redundant atomic loads in the preheader. Any suggestions on how to solve that?


https://reviews.llvm.org/D38392





More information about the llvm-commits mailing list