From: Ronan Desplanques <[email protected]>

I recently realized that Sem_Aux is in the binder's closure, unlike
Sem_Util. This patch makes it explicit in a comment since it can be
useful when choosing where to put a helper subprogram.

gcc/ada/ChangeLog:

        * sem_aux.ads: Add note.

Tested on x86_64-pc-linux-gnu, committed on master.

---
 gcc/ada/sem_aux.ads | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/gcc/ada/sem_aux.ads b/gcc/ada/sem_aux.ads
index cf80ec793ba..3bd3a466476 100644
--- a/gcc/ada/sem_aux.ads
+++ b/gcc/ada/sem_aux.ads
@@ -30,6 +30,11 @@
 --  content of entities in the tree, so this package is used for routines that
 --  require more than minimal semantic knowledge.
 
+--  However, this package is in the closure of the binder executable. If you're
+--  looking for the home of a new helper function that uses significant
+--  semantic knowledge, Sem_Util, which is not in the closure of the binder
+--  executable, will be a more convenient place.
+
 with Alloc;
 with Einfo.Entities; use Einfo.Entities;
 with Namet; use Namet;
-- 
2.53.0

Reply via email to