Re: [GSoC] Fix compiler warnings in diagnostics and analyzer - Pre-GSoC Contribution

Zeinab Ayman via Gcc <[email protected]>
Newsgroups gmane.comp.gcc.devel
Message-ID <CABMcrdh3jaZo_+iptX9ovXQTLU0iPJBkYLvqc=7gfSbFoWfi8g@mail.gmail.com>
Hi Jonathan,

Thank you for the quick clarification! I apologize for the confusion
regarding the GitHub mirror.

Since I am new to the GCC contribution workflow, I have generated a proper
patch file using git format-patch for the fixes I made in
`gcc/analyzer/engine.cc` (resolving the ambiguous exploded_cluster and
dump_args_t type mismatches).

Here is the inline patch for review:

From 66c4502ecc4254060122e212919fbdd4bc7570fd Mon Sep 17 00:00:00 2001
From: Zeinab Ayman Elshinnawy <[email protected]>
Date: Mon, 18 May 2026 16:35:10 +0300
Subject: [PATCH 11/11] analyzer: fix ambiguous exploded_cluster and
 dump_args_t type overrides

---
 gcc/analyzer/engine.cc | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/gcc/analyzer/engine.cc b/gcc/analyzer/engine.cc
index 907a36af5..de5758341 100644
--- a/gcc/analyzer/engine.cc
+++ b/gcc/analyzer/engine.cc
@@ -3927,14 +3927,14 @@ class exploded_cluster : public cluster<eg_traits>

 /* Cluster containing all exploded_node instances for one supernode. */

-class supernode_cluster : public exploded_cluster
+class supernode_cluster : public ana::exploded_cluster
 {
 public:
   supernode_cluster (const supernode *supernode) : m_supernode (supernode)
{}

   // TODO: dtor?

- void dump_dot (graphviz_out *gv, const dump_args_t &args) const final
override
+ void dump_dot (graphviz_out *gv, const dnode<ana::eg_traits>::dump_args_t
&args) const final override
   {
     gv->println ("subgraph \"cluster_supernode_%i\" {", m_supernode->m_id);
     gv->indent ();
@@ -3977,7 +3977,7 @@ private:
 /* Cluster containing all supernode_cluster instances for one
    (function, call_string) pair. */

-class function_call_string_cluster : public exploded_cluster
+class function_call_string_cluster : public ana::exploded_cluster
 {
 public:
   function_call_string_cluster (function *fun, const call_string &cs)
@@ -3991,7 +3991,7 @@ public:
        delete (*iter).second;
   }

- void dump_dot (graphviz_out *gv, const dump_args_t &args) const final
override
+ void dump_dot (graphviz_out *gv, const dnode<ana::eg_traits>::dump_args_t
&args) const final override
   {
     const char *funcname = function_name (m_fun);

@@ -4140,7 +4140,7 @@ public:
        delete (*iter).second;
   }

- void dump_dot (graphviz_out *gv, const dump_args_t &args) const final
override
+ void dump_dot (graphviz_out *gv, const dnode<ana::eg_traits>::dump_args_t
&args) const final override
   {
     int i;
     exploded_node *enode;
--
2.53.0

في الاثنين، ١٨ مايو ٢٠٢٦ ٣:٣٠ م Zeinab Ayman <[email protected]> كتب:

> Hello GCC Community and GSoC Mentors,
>
> My name is Zeinab Ayman, a Computer Engineering student preparing a
> proposal for GSoC 2026.
>
> As part of getting familiar with the GCC codebase and showcasing my
> skills, I have worked on fixing a set of compiler warnings across
> diagnostics, text-art, and analyzer modules (including -Wformat-diag,
> -Wmaybe-uninitialized, and -Wdelete-non-virtual-dtor).
>
> Since I am using GitHub to manage my workflow, I have submitted the patch
> as a Pull Request on the mirror repository here:
> https://github.com/gcc-mirror/gcc/pull/127
>
>   You can also check my GitHub profile here: https://github.com/zeinab304
>
> I would highly appreciate it if mentors could review these fixes. I am
> looking forward to contributing more and finalizing my GSoC proposal
> document.
>
> Best regards,
> Zeinab Ayman
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.