all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#42305: 28.0.50; Compilation warnings with mini-gmp
@ 2020-07-10  7:20 Eli Zaretskii
  2020-10-17 10:04 ` Lars Ingebrigtsen
  0 siblings, 1 reply; 9+ messages in thread
From: Eli Zaretskii @ 2020-07-10  7:20 UTC (permalink / raw)
  To: 42305

The configuration with mini-gmp, when GMP is not installed, doesn't
compile cleanly.  First, compilation in lib/ emits this warning:

    CC       mini-gmp-gnulib.o
  mini-gmp-gnulib.c:28:10: warning: unknown option after ‘#pragma GCC diagnostic’ kind [-Wpragmas]
  # pragma GCC diagnostic ignored "-Wsuggest-attribute=malloc"

Next, compilation in src/ yields further warnings:

  data.c: In function ‘bignum_arith_driver’:
  data.c:2811:9: warning: assignment from incompatible pointer type [enabled by default]
     accum = &mpz[0];
	   ^
  data.c:2833:13: warning: assignment from incompatible pointer type [enabled by default]
	 accum = &mpz[0];
	       ^
  data.c: In function ‘Flogcount’:
  data.c:3147:11: warning: assignment from incompatible pointer type [enabled by default]
      nonneg = &mpz[0];
	     ^

  floatfns.c: In function ‘rescale_for_division’:
  floatfns.c:374:10: warning: assignment from incompatible pointer type [enabled by default]
	 pn = t;
	    ^
  floatfns.c:382:10: warning: assignment from incompatible pointer type [enabled by default]
	 pn = t;
	    ^

  timefns.c: In function ‘frac_to_double’:
  timefns.c:622:9: warning: assignment from incompatible pointer type [enabled by default]
	 d = &mpz[1];
	   ^
  timefns.c:630:9: warning: assignment from incompatible pointer type [enabled by default]
	 n = &mpz[0];
	   ^
  timefns.c: In function ‘lisp_to_timespec’:
  timefns.c:923:21: warning: initialization from incompatible pointer type [enabled by default]
     mpz_t const *qt = q;
		       ^
  timefns.c: In function ‘time_arith’:
  timefns.c:1151:10: warning: assignment from incompatible pointer type [enabled by default]
      hzmin = hzmin1;
	    ^
  timefns.c: In function ‘time_cmp’:
  timefns.c:1265:10: warning: assignment from incompatible pointer type [enabled by default]
	 za = &mpz[0];
	    ^
  timefns.c:1266:10: warning: assignment from incompatible pointer type [enabled by default]
	 zb = &mpz[1];
	    ^

This is using

eliz@fencepost:~/git/emacs/trunk$ gcc --version
gcc (Ubuntu 4.8.4-2ubuntu1~14.04.4) 4.8.4
Copyright (C) 2013 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.


In GNU Emacs 28.0.50 (build 20, x86_64-pc-linux-gnu, GTK+ Version 2.24.23, cairo version 1.13.1)
 of 2020-07-10 built on fencepost.gnu.org
Repository revision: 9ade7ea7b77ec40c16deb4dff139ce7127a703e2
Repository branch: master
System Description: Trisquel GNU/Linux Belenos (7.0)

Recent messages:
For information about GNU Emacs and the GNU system, type C-h C-a.

Configured using:
 'configure --with-gif=no --with-tiff=no --with-jpeg=no --with-modules

Configured features:
XPM PNG CAIRO SOUND GSETTINGS GLIB NOTIFY INOTIFY ACL LIBSELINUX GNUTLS
LIBXML2 FREETYPE HARFBUZZ M17N_FLT LIBOTF ZLIB TOOLKIT_SCROLL_BARS GTK2
X11 XDBE XIM MODULES THREADS PDUMPER

Important settings:
  value of $LANG: en_US.UTF-8
  locale-coding-system: utf-8-unix

Major mode: Lisp Interaction

Minor modes in effect:
  tooltip-mode: t
  global-eldoc-mode: t
  eldoc-mode: t
  electric-indent-mode: t
  mouse-wheel-mode: t
  tool-bar-mode: t
  menu-bar-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t
  line-number-mode: t
  transient-mark-mode: t

Load-path shadows:
None found.

Features:
(shadow sort mail-extr emacsbug message rmc puny dired dired-loaddefs
rfc822 mml easymenu mml-sec password-cache epa derived epg epg-config
gnus-util rmail rmail-loaddefs text-property-search time-date subr-x seq
mm-decode mm-bodies mm-encode mail-parse rfc2231 mailabbrev gmm-utils
mailheader cl-loaddefs cl-lib sendmail rfc2047 rfc2045 ietf-drums
mm-util mail-prsvr mail-utils term/xterm xterm byte-opt gv bytecomp
byte-compile cconv tooltip eldoc electric uniquify ediff-hook vc-hooks
lisp-float-type mwheel term/x-win x-win term/common-win x-dnd tool-bar
dnd fontset image regexp-opt fringe tabulated-list replace newcomment
text-mode elisp-mode lisp-mode prog-mode register page tab-bar menu-bar
rfn-eshadow isearch timer select scroll-bar mouse jit-lock font-lock
syntax facemenu font-core term/tty-colors frame minibuffer cl-generic
cham georgian utf-8-lang misc-lang vietnamese tibetan thai tai-viet lao
korean japanese eucjp-ms cp51932 hebrew greek romanian slovak czech
european ethiopic indian cyrillic chinese composite charscript charprop
case-table epa-hook jka-cmpr-hook help simple abbrev obarray
cl-preloaded nadvice loaddefs button faces cus-face macroexp files
text-properties overlay sha1 md5 base64 format env code-pages mule
custom widget hashtable-print-readable backquote threads inotify
dynamic-setting system-font-setting font-render-setting cairo
move-toolbar gtk x-toolkit x multi-tty make-network-process emacs)

Memory information:
((conses 16 47162 6676)
 (symbols 48 6045 1)
 (strings 32 15344 1830)
 (string-bytes 1 501765)
 (vectors 16 7245)
 (vector-slots 8 75540 8480)
 (floats 8 22 301)
 (intervals 56 181 0)
 (buffers 992 10)
 (heap 1024 3481 920))






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

* bug#42305: 28.0.50; Compilation warnings with mini-gmp
  2020-07-10  7:20 bug#42305: 28.0.50; Compilation warnings with mini-gmp Eli Zaretskii
@ 2020-10-17 10:04 ` Lars Ingebrigtsen
  2020-10-17 11:15   ` Eli Zaretskii
  0 siblings, 1 reply; 9+ messages in thread
From: Lars Ingebrigtsen @ 2020-10-17 10:04 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 42305

Eli Zaretskii <eliz@gnu.org> writes:

> The configuration with mini-gmp, when GMP is not installed, doesn't
> compile cleanly.  First, compilation in lib/ emits this warning:
>
>     CC       mini-gmp-gnulib.o
>   mini-gmp-gnulib.c:28:10: warning: unknown option after ‘#pragma GCC diagnostic’ kind [-Wpragmas]
>   # pragma GCC diagnostic ignored "-Wsuggest-attribute=malloc"

Is this with

./configure --without-libgmp

?  If so, I'm not able to reproduce the bug on Debian bullseye.  But I
see that mini-gmp-gnulib.c was updated a couple of weeks after you
reported this.  Are you still seeing these warnings on your system?

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





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

* bug#42305: 28.0.50; Compilation warnings with mini-gmp
  2020-10-17 10:04 ` Lars Ingebrigtsen
@ 2020-10-17 11:15   ` Eli Zaretskii
  2020-10-18  8:14     ` Lars Ingebrigtsen
  0 siblings, 1 reply; 9+ messages in thread
From: Eli Zaretskii @ 2020-10-17 11:15 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: 42305

> From: Lars Ingebrigtsen <larsi@gnus.org>
> Cc: 42305@debbugs.gnu.org
> Date: Sat, 17 Oct 2020 12:04:05 +0200
> 
> Eli Zaretskii <eliz@gnu.org> writes:
> 
> > The configuration with mini-gmp, when GMP is not installed, doesn't
> > compile cleanly.  First, compilation in lib/ emits this warning:
> >
> >     CC       mini-gmp-gnulib.o
> >   mini-gmp-gnulib.c:28:10: warning: unknown option after ‘#pragma GCC diagnostic’ kind [-Wpragmas]
> >   # pragma GCC diagnostic ignored "-Wsuggest-attribute=malloc"
> 
> Is this with
> 
> ./configure --without-libgmp

No, this is on a system without GMP.  config.log says:

  configure:18948: checking gmp.h usability
  configure:18948: gcc -std=gnu11 -c  -O0 -g3     conftest.c >&5
  configure:18948: $? = 0
  configure:18948: result: yes
  configure:18948: checking gmp.h presence
  configure:18948: gcc -std=gnu11 -E     conftest.c
  configure:18948: $? = 0
  configure:18948: result: yes
  configure:18948: checking for gmp.h
  configure:18948: result: yes
  configure:18954: checking for library containing __gmpz_roinit_n
  configure:18985: gcc -std=gnu11 -o conftest  -O0 -g3       conftest.c -lX11   >&5
  /tmp/cchXnJLF.o: In function `main':
  /home/e/eliz/git/emacs/branch/conftest.c:221: undefined reference to `__gmpz_roinit_n'
  collect2: error: ld returned 1 exit status
  configure:18985: $? = 1
  ...
  configure:19002: result: no

The results should be identical, though, AFAIU.

> If so, I'm not able to reproduce the bug on Debian bullseye.

With what version of GCC?  This could be dependent on the compiler
version; mine in that case is

  gcc (Ubuntu 4.8.4-2ubuntu1~14.04.4) 4.8.4

as was reported originally.  (This is on fencepost.gnu.org, btw.)

> But I see that mini-gmp-gnulib.c was updated a couple of weeks after
> you reported this.  Are you still seeing these warnings on your
> system?

The warning in mini-gmp-gnulib.c is indeed gone, but GMP-related
warnings in our own sources still remain.  Examples:

  floatfns.c: In function ‘rescale_for_division’:
  floatfns.c:374:10: warning: assignment from incompatible pointer type [enabled by default]
	 pn = t;
	    ^
  floatfns.c:382:10: warning: assignment from incompatible pointer type [enabled by default]
	 pn = t;
	    ^
    CC       fns.o
  data.c: In function ‘bignum_arith_driver’:
  data.c:2821:9: warning: assignment from incompatible pointer type [enabled by default]
     accum = &mpz[0];
	   ^
  data.c:2843:13: warning: assignment from incompatible pointer type [enabled by default]
	 accum = &mpz[0];
	       ^
  data.c: In function ‘Flogcount’:
  data.c:3157:11: warning: assignment from incompatible pointer type [enabled by default]
      nonneg = &mpz[0];
	     ^
    CC       emacs-module.o
    CC       timefns.o
  timefns.c: In function ‘frac_to_double’:
  timefns.c:622:9: warning: assignment from incompatible pointer type [enabled by default]
	 d = &mpz[1];
	   ^
  timefns.c:630:9: warning: assignment from incompatible pointer type [enabled by default]
	 n = &mpz[0];
	   ^
  timefns.c: In function ‘lisp_to_timespec’:
  timefns.c:923:21: warning: initialization from incompatible pointer type [enabled by default]
     mpz_t const *qt = q;
		       ^
  timefns.c: In function ‘time_arith’:
  timefns.c:1151:10: warning: assignment from incompatible pointer type [enabled by default]
      hzmin = hzmin1;
	    ^
  timefns.c: In function ‘time_cmp’:
  timefns.c:1265:10: warning: assignment from incompatible pointer type [enabled by default]
	 za = &mpz[0];
	    ^
  timefns.c:1266:10: warning: assignment from incompatible pointer type [enabled by default]
	 zb = &mpz[1];
	    ^





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

* bug#42305: 28.0.50; Compilation warnings with mini-gmp
  2020-10-17 11:15   ` Eli Zaretskii
@ 2020-10-18  8:14     ` Lars Ingebrigtsen
  2020-10-21 18:04       ` Andy Moreton
  0 siblings, 1 reply; 9+ messages in thread
From: Lars Ingebrigtsen @ 2020-10-18  8:14 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 42305

Eli Zaretskii <eliz@gnu.org> writes:

> No, this is on a system without GMP.  config.log says:

Ah, I don't have any systems without GMP I think, so I can't test this
any further...

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





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

* bug#42305: 28.0.50; Compilation warnings with mini-gmp
  2020-10-18  8:14     ` Lars Ingebrigtsen
@ 2020-10-21 18:04       ` Andy Moreton
  2020-10-22 11:48         ` Lars Ingebrigtsen
  0 siblings, 1 reply; 9+ messages in thread
From: Andy Moreton @ 2020-10-21 18:04 UTC (permalink / raw)
  To: 42305

On Sun 18 Oct 2020, Lars Ingebrigtsen wrote:

> Eli Zaretskii <eliz@gnu.org> writes:
>
>> No, this is on a system without GMP.  config.log says:
>
> Ah, I don't have any systems without GMP I think, so I can't test this
> any further...

Does "configure --without-libgmp" help ?

    AndyM






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

* bug#42305: 28.0.50; Compilation warnings with mini-gmp
  2020-10-21 18:04       ` Andy Moreton
@ 2020-10-22 11:48         ` Lars Ingebrigtsen
  2020-10-22 13:23           ` Eli Zaretskii
  0 siblings, 1 reply; 9+ messages in thread
From: Lars Ingebrigtsen @ 2020-10-22 11:48 UTC (permalink / raw)
  To: Andy Moreton; +Cc: 42305

Andy Moreton <andrewjmoreton@gmail.com> writes:

> Does "configure --without-libgmp" help ?

That's what I tried, and Eli said "no":

> > Is this with
> > 
> > ./configure --without-libgmp
>
> No, this is on a system without GMP.  config.log says:


-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





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

* bug#42305: 28.0.50; Compilation warnings with mini-gmp
  2020-10-22 11:48         ` Lars Ingebrigtsen
@ 2020-10-22 13:23           ` Eli Zaretskii
  2020-10-22 13:24             ` Lars Ingebrigtsen
  0 siblings, 1 reply; 9+ messages in thread
From: Eli Zaretskii @ 2020-10-22 13:23 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: 42305, andrewjmoreton

> From: Lars Ingebrigtsen <larsi@gnus.org>
> Date: Thu, 22 Oct 2020 13:48:58 +0200
> Cc: 42305@debbugs.gnu.org
> 
> Andy Moreton <andrewjmoreton@gmail.com> writes:
> 
> > Does "configure --without-libgmp" help ?
> 
> That's what I tried, and Eli said "no":

No, I said it _should_ produce the same results.

But I think the other factor here is GCC.





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

* bug#42305: 28.0.50; Compilation warnings with mini-gmp
  2020-10-22 13:23           ` Eli Zaretskii
@ 2020-10-22 13:24             ` Lars Ingebrigtsen
  2020-10-22 13:37               ` Eli Zaretskii
  0 siblings, 1 reply; 9+ messages in thread
From: Lars Ingebrigtsen @ 2020-10-22 13:24 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 42305, andrewjmoreton

Eli Zaretskii <eliz@gnu.org> writes:

>> From: Lars Ingebrigtsen <larsi@gnus.org>
>> Date: Thu, 22 Oct 2020 13:48:58 +0200
>> Cc: 42305@debbugs.gnu.org
>> 
>> Andy Moreton <andrewjmoreton@gmail.com> writes:
>> 
>> > Does "configure --without-libgmp" help ?
>> 
>> That's what I tried, and Eli said "no":
>
> No, I said it _should_ produce the same results.
>
> But I think the other factor here is GCC.

Sorry, I misunderstood you.  My gcc version is:

larsi@xo:~/src/emacs/native-compile$ gcc --version
gcc (Debian 10.2.0-13) 10.2.0


-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





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

* bug#42305: 28.0.50; Compilation warnings with mini-gmp
  2020-10-22 13:24             ` Lars Ingebrigtsen
@ 2020-10-22 13:37               ` Eli Zaretskii
  0 siblings, 0 replies; 9+ messages in thread
From: Eli Zaretskii @ 2020-10-22 13:37 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: 42305, andrewjmoreton

> From: Lars Ingebrigtsen <larsi@gnus.org>
> Cc: andrewjmoreton@gmail.com,  42305@debbugs.gnu.org
> Date: Thu, 22 Oct 2020 15:24:47 +0200
> 
> > No, I said it _should_ produce the same results.
> >
> > But I think the other factor here is GCC.
> 
> Sorry, I misunderstood you.  My gcc version is:
> 
> larsi@xo:~/src/emacs/native-compile$ gcc --version
> gcc (Debian 10.2.0-13) 10.2.0

Yes.  The one which gave me trouble was 4.8...





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

end of thread, other threads:[~2020-10-22 13:37 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-07-10  7:20 bug#42305: 28.0.50; Compilation warnings with mini-gmp Eli Zaretskii
2020-10-17 10:04 ` Lars Ingebrigtsen
2020-10-17 11:15   ` Eli Zaretskii
2020-10-18  8:14     ` Lars Ingebrigtsen
2020-10-21 18:04       ` Andy Moreton
2020-10-22 11:48         ` Lars Ingebrigtsen
2020-10-22 13:23           ` Eli Zaretskii
2020-10-22 13:24             ` Lars Ingebrigtsen
2020-10-22 13:37               ` Eli Zaretskii

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.