[PATCH] D17471: [NVPTX] Annotate param loads/stores as mayLoad/mayStore.

Justin Lebar via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 29 18:01:04 PST 2016


jlebar added a comment.

In http://reviews.llvm.org/D17471#364891, @jholewinski wrote:

> Sorry, missed the original notifications.
>
> Do you have a particular example that shows the issue you're addressing here?  The pre-call stores and post-call loads should be properly chained to prevent invalid reordering.


We're seeing some (non-param) loads suspiciously being sunk.  This may be unrelated, but it's helpful to rule out...


http://reviews.llvm.org/D17471





More information about the llvm-commits mailing list