Skip to content

Commit 03aa01a

Browse files
committed
self-review: use proper float comparison
1 parent 5cc8aaa commit 03aa01a

File tree

1 file changed

+12
-12
lines changed

1 file changed

+12
-12
lines changed

clang/unittests/Parse/ParseHLSLRootSignatureTest.cpp

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -247,7 +247,7 @@ TEST_F(ParseHLSLRootSignatureTest, ValidParseStaticSamplerTest) {
247247
ASSERT_TRUE(std::holds_alternative<StaticSampler>(Elem));
248248
ASSERT_EQ(std::get<StaticSampler>(Elem).Reg.ViewType, RegisterType::SReg);
249249
ASSERT_EQ(std::get<StaticSampler>(Elem).Reg.Number, 0u);
250-
ASSERT_EQ(std::get<StaticSampler>(Elem).MipLODBias, 0.f);
250+
ASSERT_FLOAT_EQ(std::get<StaticSampler>(Elem).MipLODBias, 0.f);
251251

252252
ASSERT_TRUE(Consumer->isSatisfied());
253253
}
@@ -284,47 +284,47 @@ TEST_F(ParseHLSLRootSignatureTest, ValidParseFloatsTest) {
284284

285285
RootElement Elem = Elements[0];
286286
ASSERT_TRUE(std::holds_alternative<StaticSampler>(Elem));
287-
ASSERT_EQ(std::get<StaticSampler>(Elem).MipLODBias, 0.f);
287+
ASSERT_FLOAT_EQ(std::get<StaticSampler>(Elem).MipLODBias, 0.f);
288288

289289
Elem = Elements[1];
290290
ASSERT_TRUE(std::holds_alternative<StaticSampler>(Elem));
291-
ASSERT_EQ(std::get<StaticSampler>(Elem).MipLODBias, 1.f);
291+
ASSERT_FLOAT_EQ(std::get<StaticSampler>(Elem).MipLODBias, 1.f);
292292

293293
Elem = Elements[2];
294294
ASSERT_TRUE(std::holds_alternative<StaticSampler>(Elem));
295-
ASSERT_EQ(std::get<StaticSampler>(Elem).MipLODBias, -1.f);
295+
ASSERT_FLOAT_EQ(std::get<StaticSampler>(Elem).MipLODBias, -1.f);
296296

297297
Elem = Elements[3];
298298
ASSERT_TRUE(std::holds_alternative<StaticSampler>(Elem));
299-
ASSERT_EQ(std::get<StaticSampler>(Elem).MipLODBias, 42.f);
299+
ASSERT_FLOAT_EQ(std::get<StaticSampler>(Elem).MipLODBias, 42.f);
300300

301301
Elem = Elements[4];
302302
ASSERT_TRUE(std::holds_alternative<StaticSampler>(Elem));
303-
ASSERT_EQ(std::get<StaticSampler>(Elem).MipLODBias, 4.2f);
303+
ASSERT_FLOAT_EQ(std::get<StaticSampler>(Elem).MipLODBias, 4.2f);
304304

305305
Elem = Elements[5];
306306
ASSERT_TRUE(std::holds_alternative<StaticSampler>(Elem));
307-
ASSERT_EQ(std::get<StaticSampler>(Elem).MipLODBias, -.42f);
307+
ASSERT_FLOAT_EQ(std::get<StaticSampler>(Elem).MipLODBias, -.42f);
308308

309309
Elem = Elements[6];
310310
ASSERT_TRUE(std::holds_alternative<StaticSampler>(Elem));
311-
ASSERT_EQ(std::get<StaticSampler>(Elem).MipLODBias, 420.f);
311+
ASSERT_FLOAT_EQ(std::get<StaticSampler>(Elem).MipLODBias, 420.f);
312312

313313
Elem = Elements[7];
314314
ASSERT_TRUE(std::holds_alternative<StaticSampler>(Elem));
315-
ASSERT_EQ(std::get<StaticSampler>(Elem).MipLODBias, 0.000000000042f);
315+
ASSERT_FLOAT_EQ(std::get<StaticSampler>(Elem).MipLODBias, 0.000000000042f);
316316

317317
Elem = Elements[8];
318318
ASSERT_TRUE(std::holds_alternative<StaticSampler>(Elem));
319-
ASSERT_EQ(std::get<StaticSampler>(Elem).MipLODBias, 42.f);
319+
ASSERT_FLOAT_EQ(std::get<StaticSampler>(Elem).MipLODBias, 42.f);
320320

321321
Elem = Elements[9];
322322
ASSERT_TRUE(std::holds_alternative<StaticSampler>(Elem));
323-
ASSERT_EQ(std::get<StaticSampler>(Elem).MipLODBias, 4.2f);
323+
ASSERT_FLOAT_EQ(std::get<StaticSampler>(Elem).MipLODBias, 4.2f);
324324

325325
Elem = Elements[10];
326326
ASSERT_TRUE(std::holds_alternative<StaticSampler>(Elem));
327-
ASSERT_EQ(std::get<StaticSampler>(Elem).MipLODBias, 420000000000.f);
327+
ASSERT_FLOAT_EQ(std::get<StaticSampler>(Elem).MipLODBias, 420000000000.f);
328328

329329
Elem = Elements[11];
330330
ASSERT_TRUE(std::holds_alternative<StaticSampler>(Elem));

0 commit comments

Comments
 (0)