diff options
author | Johannes Doerfert <johannes@jdoerfert.de> | 2020-02-09 20:21:56 -0600 |
---|---|---|
committer | Johannes Doerfert <johannes@jdoerfert.de> | 2020-02-10 00:55:10 -0600 |
commit | 63adbb9a0ed9a40fc47c72502754d88d04ede1e0 (patch) | |
tree | 68ac3f0f723753cd8234e7c70945c0c6c461274e /llvm/lib | |
parent | 7e7e6594b3450dbdaffbbeb39e832f6f39ce45a9 (diff) | |
download | llvm-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.cpp | 4 |
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; } |