diff options
author | Daniel Neilson <dneilson@azul.com> | 2017-06-29 14:17:50 +0000 |
---|---|---|
committer | Daniel Neilson <dneilson@azul.com> | 2017-06-29 14:17:50 +0000 |
commit | 82b016672dcf7c1f4744dc387c816e1c6f0baba0 (patch) | |
tree | d6037d090031dde2e2355c15501e6b75bbd8e565 /lldb/source/Plugins/ScriptInterpreter/Python | |
parent | 6989caa9319f407ab66ddda3313bb7e0b7f0180f (diff) | |
download | bcm5719-llvm-82b016672dcf7c1f4744dc387c816e1c6f0baba0.tar.gz bcm5719-llvm-82b016672dcf7c1f4744dc387c816e1c6f0baba0.zip |
Explicitly check for presence of correct results in instcombine memmove test
Summary:
Rather than testing for expected results, test/Transforms/InstCombine/memmove.ll is testing for the absence of calls to llvm.memmove.
In the case of test3, the test has stopped testing for materialization of loads/stores, but remained passing due to testing for the absence of llvm.memset calls rather than the presence of load/store instructions. Right now this test ends up with an empty function body because the alloca is eliminated as safe-to-remove, which results in the llvm.memmove calls being eliminated due to a pointer arg being undef; so it is not testing for conversion of llvm.memmove into load/store instructions at all.
Reviewers: eli.friedman, anna, efriedma
Reviewed By: efriedma
Subscribers: efriedma, llvm-commits
Differential Revision: https://reviews.llvm.org/D34645
llvm-svn: 306679
Diffstat (limited to 'lldb/source/Plugins/ScriptInterpreter/Python')
0 files changed, 0 insertions, 0 deletions