base: Fix storage params safe_cast
authorDaniel R. Carvalho <odanrc@yahoo.com.br>
Wed, 3 Feb 2021 21:15:10 +0000 (18:15 -0300)
committerDaniel Carvalho <odanrc@yahoo.com.br>
Fri, 5 Feb 2021 18:51:43 +0000 (18:51 +0000)
Although they provide the exact same behavior, the params
created in the tests did not have the type expected by the
internal safe cast.

The following error was triggered:

storage.test.debug: build/NULL/base/cast.hh:47: T safe_cast(U)
 [with T = const Stats::SampleStor::Params*;
  U = const Stats::StorageParams*]:
 Assertion `ret' failed.

Change-Id: I4f2ba51f3ccdb44589e61f235997245e7d9bf3c9
Signed-off-by: Daniel R. Carvalho <odanrc@yahoo.com.br>
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/40555
Tested-by: kokoro <noreply+kokoro@google.com>
Reviewed-by: Bobby R. Bruce <bbruce@ucdavis.edu>
Maintainer: Bobby R. Bruce <bbruce@ucdavis.edu>

src/base/stats/storage.test.cc

index 717c881a88a0622735739c070cdbf6e38156d84a..8a4f6ed5e05091a5b9e058cceab00ec160d88368 100644 (file)
@@ -1083,7 +1083,7 @@ TEST(StatsSampleStorTest, SamplePrepare)
     Stats::Counter val;
     Stats::DistData data;
     Stats::DistData expected_data;
-    Stats::DistParams params(Stats::Deviation);
+    Stats::SampleStor::Params params;
     MockInfo info(&params);
 
     // Simple test with one value being sampled
@@ -1133,7 +1133,7 @@ TEST(StatsSampleStorTest, Size)
     Stats::Counter val = 10;
     Stats::Counter num_samples = 5;
     Stats::DistData data;
-    Stats::DistParams params(Stats::Deviation);
+    Stats::SampleStor::Params params;
     MockInfo info(&params);
 
     ASSERT_EQ(stor.size(), 1);
@@ -1177,7 +1177,7 @@ TEST(StatsAvgSampleStorTest, SamplePrepare)
     Stats::Counter val;
     Stats::DistData data;
     Stats::DistData expected_data;
-    Stats::DistParams params(Stats::Deviation);
+    Stats::AvgSampleStor::Params params;
     MockInfo info(&params);
 
     // Simple test with one value being sampled
@@ -1228,7 +1228,7 @@ TEST(StatsAvgSampleStorTest, Size)
     Stats::Counter val = 10;
     Stats::Counter num_samples = 5;
     Stats::DistData data;
-    Stats::DistParams params(Stats::Deviation);
+    Stats::AvgSampleStor::Params params;
     MockInfo info(&params);
 
     ASSERT_EQ(stor.size(), 1);