diff options
| author | Markus Böck <markus.bock+llvm@nextsilicon.com> | 2023-08-09 16:15:17 +0200 |
|---|---|---|
| committer | Markus Böck <markus.bock+llvm@nextsilicon.com> | 2023-08-10 12:38:54 +0200 |
| commit | 5b29f86b42bab223ed8b9f2738802138aade74e0 (patch) | |
| tree | 361d6e2a00db0d79daf5f2d47d7760fd784fd270 /libcxx/src/algorithm.cpp | |
| parent | 8dd6c7aa2ce8a97ee6af99d7bf5aac34c20975c0 (diff) | |
| download | llvm-5b29f86b42bab223ed8b9f2738802138aade74e0.zip llvm-5b29f86b42bab223ed8b9f2738802138aade74e0.tar.gz llvm-5b29f86b42bab223ed8b9f2738802138aade74e0.tar.bz2 | |
[mlir] Fix verifier of `RegionBranchOpInterface`
The verifier incorrectly passed the region number of the predecessor region instead of the successor region to `getSuccessorOperands`. This went unnoticed since all upstream `RegionBranchTerminatorOpInterface` implementations did not make use of the `index` parameter.
Adding an assert to e.g. `scf.condition` to make sure the index is valid or adding a region terminator that passes different operands to different successors immediately causes the verifier to fail as it suddenly gets incorrect types.
This patch fixes the implementation to correctly pass the successor region index.
Differential Revision: https://reviews.llvm.org/D157507
Diffstat (limited to 'libcxx/src/algorithm.cpp')
0 files changed, 0 insertions, 0 deletions