]> granicus.if.org Git - llvm/commit
Explicitly check for presence of correct results in instcombine memmove test
authorDaniel Neilson <dneilson@azul.com>
Thu, 29 Jun 2017 14:17:50 +0000 (14:17 +0000)
committerDaniel Neilson <dneilson@azul.com>
Thu, 29 Jun 2017 14:17:50 +0000 (14:17 +0000)
commit2e8a8343803ba1985e1b31307893f9eaae009138
treebd49b53e4ad3df0a3f0d65e7d5b4cb393c50ceb5
parent16d661a030cd869b049524422584dcabf89fdfb2
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

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@306679 91177308-0d34-0410-b5e6-96231b3b80d8
test/Transforms/InstCombine/memmove.ll