[PATCH] D32762: [Atomics] Let emitTrailingFence take the loaded value. NFC.

[Tim Shen via Phabricator via llvm-commits]

timshen added a comment.

In https://reviews.llvm.org/D32762#746760, @kbarton wrote:

>   D32763 only handles the trailing fences. Is there going to be another patch that deals with the leading fences as well?


No, because the reason trailing fence can depend on the loaded value is that, trailing fence is after the load in program order. IIRC data/control dependencies follows the program order.


https://reviews.llvm.org/D32762