SuccessChanges

Summary

  1. [Analyzer] Use a reference in a range-based for (details)
  2. [CodeGen] Fix getArrayAccessFor crashes as in bug 32534 with (details)
Commit 964842861c8acd53b8df8799f7c3800c5528fb72 by koraq
[Analyzer] Use a reference in a range-based for
Let the checkers use a reference instead of a copy in a range-based for
loop.
This avoids new warnings due to D68912 adds -Wrange-loop-analysis to
-Wall.
Differential Revision: https://reviews.llvm.org/D70047
The file was modifiedclang/lib/StaticAnalyzer/Core/CheckerManager.cpp
The file was modifiedclang/include/clang/StaticAnalyzer/Core/CheckerManager.h
Commit 0aff3174dcc00516fd26c6554266b05a236166bb by llvm-project
[CodeGen] Fix getArrayAccessFor crashes as in bug 32534 with
-polly-vectorizer=polly.
Root cause is VectorBlockGenerator::copyStmt iterates all instructions
in basic block, however some load instructions may be not unnecessary
thus removed by simplification. As a result, these load instructions
don't have a corresponding array.
Looking at BlockGenerator::copyBB, it only iterates instructions list of
ScopStmt. Given it must be a block type scop in case of vectorization, I
think we should do the same in VectorBlockGenerator::copyStmt.
Patch by bin.narwal <bin.narwal@gmail.com>
Differential Revision: https://reviews.llvm.org/D70076
The file was modifiedpolly/lib/CodeGen/BlockGenerators.cpp