public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug libgcc/63772] New: Double included .h files in gcc/gtype-desc.c
@ 2014-11-07 16:50 dilyan.palauzov at aegee dot org
  2014-11-07 16:56 ` [Bug middle-end/63772] " pinskia at gcc dot gnu.org
  2021-08-28 19:28 ` pinskia at gcc dot gnu.org
  0 siblings, 2 replies; 3+ messages in thread
From: dilyan.palauzov at aegee dot org @ 2014-11-07 16:50 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63772

            Bug ID: 63772
           Summary: Double included .h files in gcc/gtype-desc.c
           Product: gcc
           Version: 5.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: libgcc
          Assignee: unassigned at gcc dot gnu.org
          Reporter: dilyan.palauzov at aegee dot org

The program deheader states correctly, that gcc/gtype-desc.c includes twice
basic-block.h and ggc.h .  

This is probably caused by the fact, that
gcc/gengtype.c:open_base_files():static const char *const ifiles[] contains
twice the aforementioned .h file.

Please make sure, that gcc/gtype-desc.c does not include twice basic-block.h
and ggc.h , as one inclusion is sufficient.


^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Bug middle-end/63772] Double included .h files in gcc/gtype-desc.c
  2014-11-07 16:50 [Bug libgcc/63772] New: Double included .h files in gcc/gtype-desc.c dilyan.palauzov at aegee dot org
@ 2014-11-07 16:56 ` pinskia at gcc dot gnu.org
  2021-08-28 19:28 ` pinskia at gcc dot gnu.org
  1 sibling, 0 replies; 3+ messages in thread
From: pinskia at gcc dot gnu.org @ 2014-11-07 16:56 UTC (permalink / raw)
  To: gcc-bugs

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="UTF-8", Size: 3556 bytes --]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63772

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|libgcc                      |middle-end

--- Comment #1 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
(In reply to Дилян Палаузов from comment #0)
> Please make sure, that gcc/gtype-desc.c does not include twice basic-block.h
> and ggc.h , as one inclusion is sufficient.

It does not hurt anything and there are optimizations inside libcpp just for
this thing.
>From gcc-bugs-return-465998-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org Fri Nov 07 17:09:05 2014
Return-Path: <gcc-bugs-return-465998-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org>
Delivered-To: listarch-gcc-bugs@gcc.gnu.org
Received: (qmail 30264 invoked by alias); 7 Nov 2014 17:09:05 -0000
Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
List-Id: <gcc-bugs.gcc.gnu.org>
List-Archive: <http://gcc.gnu.org/ml/gcc-bugs/>
List-Post: <mailto:gcc-bugs@gcc.gnu.org>
List-Help: <mailto:gcc-bugs-help@gcc.gnu.org>
Sender: gcc-bugs-owner@gcc.gnu.org
Delivered-To: mailing list gcc-bugs@gcc.gnu.org
Received: (qmail 30215 invoked by uid 48); 7 Nov 2014 17:09:01 -0000
From: "jakub at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug c/59708] clang-compatible checked arithmetic builtins
Date: Fri, 07 Nov 2014 17:09:00 -0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: c
X-Bugzilla-Version: unknown
X-Bugzilla-Keywords:
X-Bugzilla-Severity: enhancement
X-Bugzilla-Who: jakub at gcc dot gnu.org
X-Bugzilla-Status: ASSIGNED
X-Bugzilla-Priority: P3
X-Bugzilla-Assigned-To: jakub at gcc dot gnu.org
X-Bugzilla-Target-Milestone: ---
X-Bugzilla-Flags:
X-Bugzilla-Changed-Fields: attachments.isobsolete attachments.created
Message-ID: <bug-59708-4-Hb8VkSVULj@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-59708-4@http.gcc.gnu.org/bugzilla/>
References: <bug-59708-4@http.gcc.gnu.org/bugzilla/>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/
Auto-Submitted: auto-generated
MIME-Version: 1.0
X-SW-Source: 2014-11/txt/msg00470.txt.bz2
Content-length: 1093

https://gcc.gnu.org/bugzilla/show_bug.cgi?idY708

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #33912|0                           |1
        is obsolete|                            |

--- Comment #14 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Created attachment 33917
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id3917&actioníit
gcc5-pr59708-wip4.patch

Hopefully feature complete version now, have added handling of result types
narrower than operand types, handling of different sign multiplications,
further optimizations to determine always non-negative or always negative
arguments (using cast analysis, VRP saved ranges), minimum precision needed to
hold all values of argument, fixed up VRP optimizations of these builtins and
added gimple_fold optimization to fold the builtins if the overflow part of the
result is never used.  Now just need to add lots of further testcases and fix
any bugs those could uncover.


^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Bug middle-end/63772] Double included .h files in gcc/gtype-desc.c
  2014-11-07 16:50 [Bug libgcc/63772] New: Double included .h files in gcc/gtype-desc.c dilyan.palauzov at aegee dot org
  2014-11-07 16:56 ` [Bug middle-end/63772] " pinskia at gcc dot gnu.org
@ 2021-08-28 19:28 ` pinskia at gcc dot gnu.org
  1 sibling, 0 replies; 3+ messages in thread
From: pinskia at gcc dot gnu.org @ 2021-08-28 19:28 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63772

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED
   Target Milestone|---                         |6.0

--- Comment #2 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
basic-block.h was removed at r6-2054
ggc.h was removed at r6-1445

So all fixed.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-08-28 19:28 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-11-07 16:50 [Bug libgcc/63772] New: Double included .h files in gcc/gtype-desc.c dilyan.palauzov at aegee dot org
2014-11-07 16:56 ` [Bug middle-end/63772] " pinskia at gcc dot gnu.org
2021-08-28 19:28 ` pinskia at gcc dot gnu.org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).