From 3ef9dfdd425e061a71dd5ca168c2dfec03699754 Mon Sep 17 00:00:00 2001 From: oli Date: Sun, 1 Nov 2020 17:00:09 +0000 Subject: [PATCH] Update comment --- compiler/rustc_middle/src/ty/consts/int.rs | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/compiler/rustc_middle/src/ty/consts/int.rs b/compiler/rustc_middle/src/ty/consts/int.rs index bdb6e4e9660..4b842b75461 100644 --- a/compiler/rustc_middle/src/ty/consts/int.rs +++ b/compiler/rustc_middle/src/ty/consts/int.rs @@ -165,9 +165,8 @@ impl ScalarInt { } /// Make sure the `data` fits in `size`. - /// This is guaranteed by all constructors here, but since the enum variants are public, - /// it could still be violated (even though no code outside this file should - /// construct `Scalar`s). + /// This is guaranteed by all constructors here, but having had this check saved us from + /// bugs many times in the past, so keeping it around is definitely worth it. #[inline(always)] fn check_data(self) { // Using a block `{self.data}` here to force a copy instead of using `self.data`