diff options
| author | George Burgess IV <george.burgess.iv@gmail.com> | 2018-08-10 05:14:43 +0000 |
|---|---|---|
| committer | George Burgess IV <george.burgess.iv@gmail.com> | 2018-08-10 05:14:43 +0000 |
| commit | ff08c80efcd5f69b9ac9881699b45fb8aa10f15e (patch) | |
| tree | 8994606ea6ce9320ea9aab94e2039ba82ef933f1 /clang/lib/CodeGen/CodeGenModule.cpp | |
| parent | 909889b2cbd55fbc4a5bb7f155bd286a82da1fef (diff) | |
| download | llvm-ff08c80efcd5f69b9ac9881699b45fb8aa10f15e.zip llvm-ff08c80efcd5f69b9ac9881699b45fb8aa10f15e.tar.gz llvm-ff08c80efcd5f69b9ac9881699b45fb8aa10f15e.tar.bz2 | |
[MemorySSA] "Fix" lifetime intrinsic handling
MemorySSA currently creates MemoryAccesses for lifetime intrinsics, and
sometimes treats them as clobbers. This may/may not be the best way
forward, but while we're doing it, we should consider
MayAlias/PartialAlias to be clobbers.
The ideal fix here is probably to remove all of this reasoning about
lifetimes from MemorySSA + put it into the passes that need to care. But
that's a wayyy broader fix that needs some consensus, and we have
miscompiles + a release branch today, and this should solve the
miscompiles just as well.
differential revision is D43269. Landing without an explicit LGTM (and
without using the special please-autoclose-this syntax) so we can still
use that revision as a place to decide what the right fix here is.
llvm-svn: 339411
Diffstat (limited to 'clang/lib/CodeGen/CodeGenModule.cpp')
0 files changed, 0 insertions, 0 deletions