diff options
| author | Philip Reames <listmail@philipreames.com> | 2018-09-10 16:00:27 +0000 |
|---|---|---|
| committer | Philip Reames <listmail@philipreames.com> | 2018-09-10 16:00:27 +0000 |
| commit | 5660bd460b0f8250be31ee470e669f57557130c9 (patch) | |
| tree | 3a18495fa35e28e23ddccb56e786720cae68394b /llvm/lib/Analysis | |
| parent | ce4dd0ae38378b30281e24cd0d2126e6912eb110 (diff) | |
| download | bcm5719-llvm-5660bd460b0f8250be31ee470e669f57557130c9.tar.gz bcm5719-llvm-5660bd460b0f8250be31ee470e669f57557130c9.zip | |
[AST] Visit memtransfer arguments in order
The only point to this change is the test diffs. When I remove this code entirely (in favor of the recently added generic handling), I don't want there to be any confusion due to spurious test diffs.
As an aside, the fact out tests are AST construction order dependent is not great. I thought about fixing that, but the reasonable schemes I might want (e.g. sort by name) need the test diffs anyways.
Philip
llvm-svn: 341841
Diffstat (limited to 'llvm/lib/Analysis')
| -rw-r--r-- | llvm/lib/Analysis/AliasSetTracker.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Analysis/AliasSetTracker.cpp b/llvm/lib/Analysis/AliasSetTracker.cpp index d0997932260..0d0277e9c34 100644 --- a/llvm/lib/Analysis/AliasSetTracker.cpp +++ b/llvm/lib/Analysis/AliasSetTracker.cpp @@ -407,8 +407,8 @@ void AliasSetTracker::add(AnyMemSetInst *MSI) { } void AliasSetTracker::add(AnyMemTransferInst *MTI) { - addPointer(MemoryLocation::getForSource(MTI), AliasSet::RefAccess); addPointer(MemoryLocation::getForDest(MTI), AliasSet::ModAccess); + addPointer(MemoryLocation::getForSource(MTI), AliasSet::RefAccess); } void AliasSetTracker::addUnknown(Instruction *Inst) { |
