diff options
author | Nuno Lopes <nuno.lopes@tecnico.ulisboa.pt> | 2022-01-26 10:10:22 +0000 |
---|---|---|
committer | Nuno Lopes <nuno.lopes@tecnico.ulisboa.pt> | 2022-01-26 10:19:18 +0000 |
commit | 24a49e99f386432fc6fa46faf6e2ba91cfaed2df (patch) | |
tree | 00c5928341af30990408cfcea8799d14befcbecf /llvm/lib/TextAPI/Architecture.cpp | |
parent | 66c602be25c15ca69f6c3a618427ba0237c0d4a9 (diff) | |
download | llvm-24a49e99f386432fc6fa46faf6e2ba91cfaed2df.zip llvm-24a49e99f386432fc6fa46faf6e2ba91cfaed2df.tar.gz llvm-24a49e99f386432fc6fa46faf6e2ba91cfaed2df.tar.bz2 |
[NewGVN] FIx phi-of-ops in the presence of memory read operations
The phi-of-ops functionality has a function OpIsSafeForPHIOfOps
to determine when it's safe to create the new phi.
But this function only checks for the obvious dominator conditions
and ignores memory.
This patch takes the conservative approach and disables phi-of-ops
whenever there's a load that doesn't dominate the phi, as its
value may be affected by a store inside the loop.
This can be improved later to check aliasing between the
load/stores.
Fixes https://llvm.org/PR53277
Reviewed By: asbirlea
Differential Revision: https://reviews.llvm.org/D117999
Diffstat (limited to 'llvm/lib/TextAPI/Architecture.cpp')
0 files changed, 0 insertions, 0 deletions