mirror of
https://github.com/c64scene-ar/llvm-6502.git
synced 2025-06-14 14:24:05 +00:00
Revert commit 131781, to see if it fixes the x86-64 dragonegg buildbot.
Original log message: When BasicAA can determine that two pointers have the same base but differ by a dynamic offset, return PartialAlias instead of MayAlias. See the comment in the code for details. This fixes PR9971. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@131809 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
@ -940,17 +940,7 @@ BasicAliasAnalysis::aliasGEP(const GEPOperator *GEP1, uint64_t V1Size,
|
||||
return NoAlias;
|
||||
}
|
||||
|
||||
// Statically, we can see that the base objects are the same, but the
|
||||
// pointers have dynamic offsets which we can't resolve. And none of our
|
||||
// little tricks above worked.
|
||||
//
|
||||
// TODO: Returning PartialAlias instead of MayAlias is a mild hack; the
|
||||
// practical effect of this is protecting TBAA in the case of dynamic
|
||||
// indices into arrays of unions. An alternative way to solve this would
|
||||
// be to have clang emit extra metadata for unions and/or union accesses.
|
||||
// A union-specific solution wouldn't handle the problem for malloc'd
|
||||
// memory however.
|
||||
return PartialAlias;
|
||||
return MayAlias;
|
||||
}
|
||||
|
||||
/// aliasSelect - Provide a bunch of ad-hoc rules to disambiguate a Select
|
||||
|
Reference in New Issue
Block a user