Displaying 2 results from an estimated 2 matches for "d5422".
Did you mean:
15422
2018 Oct 02
0
Reordering of load/stores using MemorySSA
...it a try to implement it if it is straightforward.
> From which file should I start?
There have been PDSE/etc patches posted to llvm-dev before that are
probably a good start.
> By the way, while looking through memory related optimizations, I found this patch -
> https://reviews.llvm.org/D5422 .
> Would it make sense if store PRE deals with atomic operations
> as well? I wonder whether people are very interested in
> optimizing atomic operations / there are more patches like this.
>
> Thank you,
> Juneyoung Lee
>
> On Tue, Oct 2, 2018 at 1:38 PM Daniel Berlin <...
2018 Oct 02
2
Reordering of load/stores using MemorySSA
Hello all,
It seems that it is insufficient to
rely on MemorySSA to correctly reorder load/stores.
For example, we have this following code:
v = load q
store 10, p
=>
store 10, p
v = load q // This is incorrect if p and q may alias
In the MemorySSA representation however,
there's no syntactic dependency between load/store,
because MemorySSA before transformation would look like this: