diff options
| author | Tobias Grosser <tobias@grosser.es> | 2017-01-17 12:00:42 +0000 |
|---|---|---|
| committer | Tobias Grosser <tobias@grosser.es> | 2017-01-17 12:00:42 +0000 |
| commit | e1ff0cf2eb7bcd1658731866915e89da4a78ed3d (patch) | |
| tree | 064c8ad82cb8570099813ae4e0c6daba2b6ec1e9 /llvm/lib/Support/Compression.cpp | |
| parent | e5cfc671131e67768224355fcde6a1ab4affbc35 (diff) | |
| download | llvm-e1ff0cf2eb7bcd1658731866915e89da4a78ed3d.zip llvm-e1ff0cf2eb7bcd1658731866915e89da4a78ed3d.tar.gz llvm-e1ff0cf2eb7bcd1658731866915e89da4a78ed3d.tar.bz2 | |
Relax assert when setting access functions with invariant base pointers
Summary:
Instead of forbidding such access functions completely, we verify that their
base pointer has been hoisted and only assert in case the base pointer was
not hoisted.
I was trying for a little while to get a test case that ensures the assert is
correctly fired in case of invariant load hoisting being disabled, but I could
not find a good way to do so, as llvm-lit immediately aborts if a command
yields a non-zero return value. As we do not generally test our asserts,
not having a test case here seems OK.
This resolves http://llvm.org/PR31494
Suggested-by: Michael Kruse <llvm@meinersbur.de>
Reviewers: efriedma, jdoerfert, Meinersbur, gareevroman, sebpop, zinob, huihuiz, pollydev
Reviewed By: Meinersbur
Differential Revision: https://reviews.llvm.org/D28798
llvm-svn: 292213
Diffstat (limited to 'llvm/lib/Support/Compression.cpp')
0 files changed, 0 insertions, 0 deletions