From: Yannick Moy Date: Fri, 25 May 2018 09:02:58 +0000 (+0000) Subject: [Ada] Detect misplaced assertions between loop invariants X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=56ce7e4aa9cfbeb2df1a02a4ae899faaeedb2ab4;p=gcc.git [Ada] Detect misplaced assertions between loop invariants Loop invariants and loop variants should all be colocated, as defined in SPARK RM 5.5.3(8). The code checking that rule was incorrectly accepting pragma Assert between two loop invariants. Now fixed. 2018-05-25 Yannick Moy gcc/ada/ * sem_prag.adb (Check_Grouping): Modify test to ignore statements and declarations not coming from source. From-SVN: r260715 --- diff --git a/gcc/ada/ChangeLog b/gcc/ada/ChangeLog index 37dce8b5367..8fd40527f0a 100644 --- a/gcc/ada/ChangeLog +++ b/gcc/ada/ChangeLog @@ -1,3 +1,8 @@ +2018-05-25 Yannick Moy + + * sem_prag.adb (Check_Grouping): Modify test to ignore statements and + declarations not coming from source. + 2018-05-25 Fedor Rybin * doc/gnat_ugn/gnat_utility_programs.rst: Document new switch diff --git a/gcc/ada/sem_prag.adb b/gcc/ada/sem_prag.adb index cd46404cf8e..85a064d3ef2 100644 --- a/gcc/ada/sem_prag.adb +++ b/gcc/ada/sem_prag.adb @@ -5979,9 +5979,14 @@ package body Sem_Prag is Prag := Stmt; -- Skip declarations and statements generated by - -- the compiler during expansion. + -- the compiler during expansion. Note that some + -- source statements (e.g. pragma Assert) may have + -- been transformed so that they do not appear as + -- coming from source anymore, so we instead look + -- at their Original_Node. - elsif not Comes_From_Source (Stmt) then + elsif not Comes_From_Source (Original_Node (Stmt)) + then null; -- A non-pragma is separating the group from the