[PATCH] D119818: [MemoryDepndency] Relax the re-ordering with volatile store.

Serguei Katkov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 15 02:03:36 PST 2022


skatkov added a comment.

@reames , last time this part was touched (https://reviews.llvm.org/D16857, ooch 2016) you provided the meaningful review. I would appreciate if you could look at this at some point.

To be honest I do not have a motivation example for this change and probably it terms of "do not touch if it works" I can abandon this patch.
I just mentioned inconsistency in handling Load and Store and according to my understanding of spec there is no difference and decided it is better to improve it.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D119818/new/

https://reviews.llvm.org/D119818



More information about the llvm-commits mailing list