[PATCH] D49388: [DAG] Fix Memory ordering check in ReduceLoadOpStore.

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 16 10:53:39 PDT 2018


spatel added reviewers: efriedma, craig.topper, RKSimon.
spatel added a comment.

Thanks!
Adding potential reviewers (I still don't have a good understanding). Bug link:
https://bugs.llvm.org/show_bug.cgi?id=37826

It would be better to remove the main() code in the test file, auto-generate the full output showing the buggy memop sequence, and commit that against trunk. Then, when you regenerate the results after this patch, we'll see that the load and stores have changed.


Repository:
  rL LLVM

https://reviews.llvm.org/D49388





More information about the llvm-commits mailing list