public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug d/94305] New: libphobos: Add configure flag to build phobos in non-release mode
@ 2020-03-24 16:52 ibuclaw at gdcproject dot org
  2020-03-25 10:22 ` [Bug d/94305] " marxin at gcc dot gnu.org
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: ibuclaw at gdcproject dot org @ 2020-03-24 16:52 UTC (permalink / raw)
  To: gcc-bugs

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

            Bug ID: 94305
           Summary: libphobos: Add configure flag to build phobos in
                    non-release mode
           Product: gcc
           Version: 10.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: d
          Assignee: ibuclaw at gdcproject dot org
          Reporter: ibuclaw at gdcproject dot org
  Target Milestone: ---

Currently, libphobos is always built in non-release mode, which means all
asserts, contracts and invariants are compiled in.

For speed, `-frelease' should be the default, but allow this to be overridden
for testing purposes.  Maybe re-using --enable-checking?

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

* [Bug d/94305] libphobos: Add configure flag to build phobos in non-release mode
  2020-03-24 16:52 [Bug d/94305] New: libphobos: Add configure flag to build phobos in non-release mode ibuclaw at gdcproject dot org
@ 2020-03-25 10:22 ` marxin at gcc dot gnu.org
  2020-04-08 22:47 ` cvs-commit at gcc dot gnu.org
  2020-04-08 22:48 ` ibuclaw at gdcproject dot org
  2 siblings, 0 replies; 4+ messages in thread
From: marxin at gcc dot gnu.org @ 2020-03-25 10:22 UTC (permalink / raw)
  To: gcc-bugs

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

Martin Liška <marxin at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
     Ever confirmed|0                           |1
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2020-03-25
                 CC|                            |marxin at gcc dot gnu.org

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

* [Bug d/94305] libphobos: Add configure flag to build phobos in non-release mode
  2020-03-24 16:52 [Bug d/94305] New: libphobos: Add configure flag to build phobos in non-release mode ibuclaw at gdcproject dot org
  2020-03-25 10:22 ` [Bug d/94305] " marxin at gcc dot gnu.org
@ 2020-04-08 22:47 ` cvs-commit at gcc dot gnu.org
  2020-04-08 22:48 ` ibuclaw at gdcproject dot org
  2 siblings, 0 replies; 4+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2020-04-08 22:47 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #1 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Iain Buclaw <ibuclaw@gcc.gnu.org>:

https://gcc.gnu.org/g:c0dbfbd763ad77d6beaa345ce94afc93e193236e

commit r10-7644-gc0dbfbd763ad77d6beaa345ce94afc93e193236e
Author: Iain Buclaw <ibuclaw@gdcproject.org>
Date:   Thu Apr 9 00:41:14 2020 +0200

    libphobos: Add --enable-libphobos-checking configure option

    As GDCFLAGS is overriden by the top-level make file with '-O2 -g',
    libphobos ends up always being built with all contracts, invariants, and
    asserts compiled in.  This adds a new configurable that defaults to omit
    compiling any run-time checks into the library using '-frelease'.

    Other choices either set the flags '-fno-release', enabling all run-time
    checks, or '-fassert', which only compiles in asserts.

    The omission of compiling in contracts results in a smaller library
    size, with faster build times.

    libphobos/ChangeLog:

            PR d/94305
            * Makefile.in: Regenerate.
            * configure: Regenerate.
            * configure.ac: Add --enable-libphobos-checking and substitute
            CHECKING_DFLAGS.  Remove -frelease from GDCFLAGS.
            * libdruntime/Makefile.am: Add CHECKING_DFLAGS to AM_DFLAGS.
            * libdruntime/Makefile.in: Regenerate.
            * src/Makefile.am: Add CHECKING_DFLAGS to AM_DFLAGS.
            * src/Makefile.in: Regenerate.
            * testsuite/Makefile.in: Regenerate.
            * testsuite/testsuite_flags.in: Add -fno-release -funittest to
            --gdcflags.

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

* [Bug d/94305] libphobos: Add configure flag to build phobos in non-release mode
  2020-03-24 16:52 [Bug d/94305] New: libphobos: Add configure flag to build phobos in non-release mode ibuclaw at gdcproject dot org
  2020-03-25 10:22 ` [Bug d/94305] " marxin at gcc dot gnu.org
  2020-04-08 22:47 ` cvs-commit at gcc dot gnu.org
@ 2020-04-08 22:48 ` ibuclaw at gdcproject dot org
  2 siblings, 0 replies; 4+ messages in thread
From: ibuclaw at gdcproject dot org @ 2020-04-08 22:48 UTC (permalink / raw)
  To: gcc-bugs

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

Iain Buclaw <ibuclaw at gdcproject dot org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #2 from Iain Buclaw <ibuclaw at gdcproject dot org> ---
Done.

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

end of thread, other threads:[~2020-04-08 22:48 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-24 16:52 [Bug d/94305] New: libphobos: Add configure flag to build phobos in non-release mode ibuclaw at gdcproject dot org
2020-03-25 10:22 ` [Bug d/94305] " marxin at gcc dot gnu.org
2020-04-08 22:47 ` cvs-commit at gcc dot gnu.org
2020-04-08 22:48 ` ibuclaw at gdcproject dot 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).