diff options
| author | vpykhtin <valery.pykhtin@gmail.com> | 2020-10-14 14:47:18 +0300 |
|---|---|---|
| committer | vpykhtin <valery.pykhtin@gmail.com> | 2020-10-23 19:17:48 +0300 |
| commit | 00255f419298b93c812324a257ff057e57ab5c04 (patch) | |
| tree | eb72474eaeb90b7b940c41b5c60ae3ad4920adbe /clang/lib/Basic/SourceManager.cpp | |
| parent | 5dd39923a09ec284d30083e5fadaa29e618d4446 (diff) | |
| download | llvm-00255f419298b93c812324a257ff057e57ab5c04.zip llvm-00255f419298b93c812324a257ff057e57ab5c04.tar.gz llvm-00255f419298b93c812324a257ff057e57ab5c04.tar.bz2 | |
[AMDGPU] Fix access beyond the end of the basic block in execMayBeModifiedBeforeAnyUse.
I was wrong in thinking that MRI.use_instructions return unique instructions and mislead Jay in his previous patch D64393.
First loop counted more instructions than it was in reality and the second loop went beyond the basic block with that counter.
I used Jay's previous code that relied on MRI.use_operands to constrain the number of instructions to check among.
modifiesRegister is inlined to reduce the number of passes over instruction operands and added assert on BB end boundary.
Differential Revision: https://reviews.llvm.org/D89386
Diffstat (limited to 'clang/lib/Basic/SourceManager.cpp')
0 files changed, 0 insertions, 0 deletions