]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
ada: Remove repeated guards in validity checks
authorPiotr Trojanek <trojanek@adacore.com>
Thu, 8 Aug 2024 08:59:57 +0000 (10:59 +0200)
committerMarc Poulhiès <dkm@gcc.gnu.org>
Mon, 2 Sep 2024 08:22:49 +0000 (10:22 +0200)
Routine Insert_Valid_Check only applies checks when Expr_Known_Valid
query returns False; there is no need to call this query before
inserting checks.

Code cleanup; behavior is unaffected.

gcc/ada/

* exp_imgv.adb (Expand_User_Defined_Enumeration_Image)
(Expand_Image_Attribute): Remove redundant guards.

gcc/ada/exp_imgv.adb

index e5d84cc52e34560e4b34ec2ac19a317b4dd9bf6b..ce052c13c8802206590afbe10dd977a5fccfe0eb 100644 (file)
@@ -896,9 +896,7 @@ package body Exp_Imgv is
          --  Apply a validity check, since it is a bit drastic to get a
          --  completely junk image value for an invalid value.
 
-         if not Expr_Known_Valid (Expr) then
-            Insert_Valid_Check (Expr);
-         end if;
+         Insert_Valid_Check (Expr);
 
          --  Generate:
          --    P1 : constant Natural := Typ'Pos (Typ?(Expr));
@@ -1249,9 +1247,7 @@ package body Exp_Imgv is
             --  Apply a validity check, since it is a bit drastic to get a
             --  completely junk image value for an invalid value.
 
-            if not Expr_Known_Valid (Expr) then
-               Insert_Valid_Check (Expr);
-            end if;
+            Insert_Valid_Check (Expr);
 
             Enum_Case := True;
          end if;