aboutsummaryrefslogtreecommitdiff
path: root/llvm/lib
diff options
context:
space:
mode:
authorJohannes Doerfert <johannes@jdoerfert.de>2020-02-09 20:21:56 -0600
committerJohannes Doerfert <johannes@jdoerfert.de>2020-02-10 00:55:10 -0600
commit63adbb9a0ed9a40fc47c72502754d88d04ede1e0 (patch)
tree68ac3f0f723753cd8234e7c70945c0c6c461274e /llvm/lib
parent7e7e6594b3450dbdaffbbeb39e832f6f39ce45a9 (diff)
downloadllvm-63adbb9a0ed9a40fc47c72502754d88d04ede1e0.zip
llvm-63adbb9a0ed9a40fc47c72502754d88d04ede1e0.tar.gz
llvm-63adbb9a0ed9a40fc47c72502754d88d04ede1e0.tar.bz2
[Attributor][FIX] Remove FIXME that seems outdated
The change is performed as stated by the FIXME and the tests are adjusted. All changes look fine to me and values can be inferred as undef without it being an error.
Diffstat (limited to 'llvm/lib')
-rw-r--r--llvm/lib/Transforms/IPO/Attributor.cpp4
1 files changed, 1 insertions, 3 deletions
diff --git a/llvm/lib/Transforms/IPO/Attributor.cpp b/llvm/lib/Transforms/IPO/Attributor.cpp
index d8d3108..da5260b 100644
--- a/llvm/lib/Transforms/IPO/Attributor.cpp
+++ b/llvm/lib/Transforms/IPO/Attributor.cpp
@@ -4408,9 +4408,7 @@ struct AAValueSimplifyImpl : AAValueSimplify {
else
return false;
} else {
- // FIXME: It should be llvm::None but if you set llvm::None,
- // values are mistakenly infered as `undef` now.
- SimplifiedAssociatedValue = &getAssociatedValue();
+ SimplifiedAssociatedValue = llvm::None;
}
return true;
}