diff options
| author | Florian Hahn <flo@fhahn.com> | 2020-09-29 09:18:19 +0100 |
|---|---|---|
| committer | Florian Hahn <flo@fhahn.com> | 2020-09-29 09:18:19 +0100 |
| commit | b76df593eb660d1e4c9f1384a75b404ee5bcd06f (patch) | |
| tree | 504ce4ce189c5fad35ced3f203032e79b4a323bc /llvm/lib/Analysis/Loads.cpp | |
| parent | 86a20d9e34f5a9989da72097f23f3b0a44157e73 (diff) | |
| download | llvm-b76df593eb660d1e4c9f1384a75b404ee5bcd06f.zip llvm-b76df593eb660d1e4c9f1384a75b404ee5bcd06f.tar.gz llvm-b76df593eb660d1e4c9f1384a75b404ee5bcd06f.tar.bz2 | |
Revert "Recommit "[SCCP] Do not replace deref'able ptr with un-deref'able one.""
Looks like there is still another remaining issue:
http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-bootstrap-msan/builds/22273/steps/build%20libcxx%2Fmsan/logs/stdio
This reverts commit 86a20d9e34f5a9989da72097f23f3b0a44157e73.
Diffstat (limited to 'llvm/lib/Analysis/Loads.cpp')
| -rw-r--r-- | llvm/lib/Analysis/Loads.cpp | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/llvm/lib/Analysis/Loads.cpp b/llvm/lib/Analysis/Loads.cpp index 5b9f466..d63f6b9 100644 --- a/llvm/lib/Analysis/Loads.cpp +++ b/llvm/lib/Analysis/Loads.cpp @@ -510,10 +510,6 @@ bool llvm::canReplacePointersIfEqual(Value *A, Value *B, const DataLayout &DL, assert(Ty == B->getType() && Ty->isPointerTy() && "values must have matching pointer types"); - // Function pointers are not directly dereferenced using load/store - // instructions. Allow any replacements for now. - if (A->getType()->getPointerElementType()->isFunctionTy()) - return true; // NOTE: The checks in the function are incomplete and currently miss illegal // cases! The current implementation is a starting point and the // implementation should be made stricter over time. |