Skip to content

Commit 69affe7

Browse files
committed
[llvm][Analysis] Silence warning when building with MSVC
When building an assert-enabled target, silence the following: ``` C:\git\llvm-project\llvm\include\llvm/Analysis/DependenceAnalysis.h(290): warning C4018: '<=': signed/unsigned mismatch ```
1 parent addfdb5 commit 69affe7

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

llvm/include/llvm/Analysis/DependenceAnalysis.h

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -287,7 +287,8 @@ class LLVM_ABI FullDependence final : public Dependence {
287287
assert(0 < Level && Level <= Levels && "Level out of range");
288288
return DV[Level - 1];
289289
} else {
290-
assert(Levels < Level && Level <= Levels + SameSDLevels &&
290+
assert(Levels < Level &&
291+
Level <= static_cast<unsigned>(Levels) + SameSDLevels &&
291292
"isSameSD level out of range");
292293
return DVSameSD[Level - Levels - 1];
293294
}

llvm/lib/Analysis/DependenceAnalysis.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -389,7 +389,8 @@ bool FullDependence::isSplitable(unsigned Level, bool isSameSD) const {
389389
// performed across two separate loop nests that have the Same iteration space
390390
// and Depth.
391391
bool FullDependence::inSameSDLoops(unsigned Level) const {
392-
assert(0 < Level && Level <= Levels + SameSDLevels && "Level out of range");
392+
assert(0 < Level && Level <= static_cast<unsigned>(Levels) + SameSDLevels &&
393+
"Level out of range");
393394
return Level > Levels;
394395
}
395396

0 commit comments

Comments
 (0)