Skip to content

Commit a4018bb

Browse files
committed
review: fix up comments
1 parent d498922 commit a4018bb

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

llvm/lib/Frontend/HLSL/RootSignatureValidations.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -252,7 +252,7 @@ findOverlappingRanges(ArrayRef<RangeInfo> Infos) {
252252
llvm::SmallVector<OverlappingRanges> Overlaps;
253253
using GroupT = std::pair<dxil::ResourceClass, /*Space*/ uint32_t>;
254254

255-
// 3. First we will init our state to track:
255+
// First we will init our state to track:
256256
if (Infos.size() == 0)
257257
return Overlaps; // No ranges to overlap
258258
GroupT CurGroup = {Infos[0].Class, Infos[0].Space};
@@ -276,7 +276,7 @@ findOverlappingRanges(ArrayRef<RangeInfo> Infos) {
276276
Range.clear();
277277
};
278278

279-
// 3: Iterate through collected RangeInfos
279+
// Iterate through collected RangeInfos
280280
for (const RangeInfo &Info : Infos) {
281281
GroupT InfoGroup = {Info.Class, Info.Space};
282282
// Reset our ResourceRanges when we enter a new group
@@ -285,12 +285,12 @@ findOverlappingRanges(ArrayRef<RangeInfo> Infos) {
285285
CurGroup = InfoGroup;
286286
}
287287

288-
// 3A: Insert range info into corresponding Visibility ResourceRange
288+
// Insert range info into corresponding Visibility ResourceRange
289289
ResourceRange &VisRange = Ranges[llvm::to_underlying(Info.Visibility)];
290290
if (std::optional<const RangeInfo *> Overlapping = VisRange.insert(Info))
291291
Overlaps.push_back(OverlappingRanges(&Info, Overlapping.value()));
292292

293-
// 3B: Check for overlap in all overlapping Visibility ResourceRanges
293+
// Check for overlap in all overlapping Visibility ResourceRanges
294294
//
295295
// If the range that we are inserting has ShaderVisiblity::All it needs to
296296
// check for an overlap in all other visibility types as well.

0 commit comments

Comments
 (0)