From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from hamza.pair.com (hamza.pair.com [209.68.5.143]) by sourceware.org (Postfix) with ESMTPS id 0EE943858D39; Fri, 26 Nov 2021 18:03:24 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 0EE943858D39 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=pfeifer.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=pfeifer.com Received: from hamza.pair.com (localhost [127.0.0.1]) by hamza.pair.com (Postfix) with ESMTP id B69B333ED1; Fri, 26 Nov 2021 13:03:23 -0500 (EST) Received: from [192.168.1.4] (unknown [46.57.95.102]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by hamza.pair.com (Postfix) with ESMTPSA id EC7A533ECF; Fri, 26 Nov 2021 13:03:22 -0500 (EST) Date: Fri, 26 Nov 2021 19:03:26 +0100 (CET) From: Gerald Pfeifer To: David Malcolm , Antoni Boucher , Antoni Boucher via Jit cc: gcc-patches@gcc.gnu.org Subject: Re: [PATCH] libgccjit: add some reflection functions in the jit C api In-Reply-To: Message-ID: <321ec15e-2175-a6fe-5a37-8cef132c4143@pfeifer.com> References: <20200902010120.crnx55ev635ceiey@bouanto-desktop.localdomain> <54ba5c58dbd2b8c7b1f3276c2b87cddf55e258bd.camel@redhat.com> <20201103221324.xw3jbvrw2uwdc4rz@bouanto-desktop.localdomain> <3388bb8c84e68cd7b0698dc154e7a5666c0d2cde.camel@redhat.com> <0e8b6450bcb23182b85342d8010c3bea0c297ba2.camel@zoho.com> <534254132a841b75d555a52ce952f84418f168c2.camel@redhat.com> <3c328b40570b10b22a8925fd64b2781dc9264358.camel@zoho.com> <715f6ebfe1627fc39d8ec3b15f1e38f8378f3db7.camel@redhat.com> <711092bc80237e32caa48f2060c93be30cb9c220.camel@zoho.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII X-Spam-Status: No, score=-3.7 required=5.0 tests=BAYES_00, KAM_DMARC_STATUS, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org X-BeenThere: jit@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Jit mailing list List-Unsubscribe: , List-Archive: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 26 Nov 2021 18:03:25 -0000 On Fri, 19 Nov 2021, David Malcolm via Gcc-patches wrote: > On Mon, 2021-09-27 at 20:53 -0400, Antoni Boucher wrote: >> I fixed an issue (it would show an error message when >> gcc_jit_type_dyncast_function_ptr_type was called on a type different >> than a function pointer type). > The updated patch is good for trunk, assuming you re-ran the regression > tests successfully. I have received a report of GCC builds now failing on FreeBSD/i386: In function 'gcc_jit_type* gcc_jit_function_type_get_param_type(gcc_jit_function_type*, size_t)': /wrkdirs/usr/ports/lang/gcc12-devel/work/gcc-12-20211121/gcc/jit/libgccjit.c: 184:35: error: format '%ld' expects argument of type 'long int', but argument 5 has type 'size_t' {aka 'unsigned int'} [-Werror=format=] 184 | jit_error ((CTXT), (LOC), "%s: " ERR_FMT, \ | ^ 185 | __func__, (A0), (A1), (A2)); \ | ~~~~ /wrkdirs/usr/ports/lang/gcc12-devel/work/gcc-12-20211121/gcc/jit/libgccjit.c: 230:3: note: in expansion of macro 'RETURN_VAL_IF_FAIL_PRINTF3' 230 | RETURN_VAL_IF_FAIL_PRINTF3 (TEST_EXPR, NULL, CTXT, LOC, ERR_FMT, A0, A1, A2) | ^~~~~~~~~~~~~~~~~~~~~~~~~~ /wrkdirs/usr/ports/lang/gcc12-devel/work/gcc-12-20211121/gcc/jit/libgccjit.c: 708:3: note: in expansion of macro 'RETURN_NULL_IF_FAIL_PRINTF3' 708 | RETURN_NULL_IF_FAIL_PRINTF3 (index < num_params, | ^~~~~~~~~~~~~~~~~~~~~~~~~~~ /wrkdirs/usr/ports/lang/gcc12-devel/work/gcc-12-20211121/gcc/jit/libgccjit.c: 710:44: note: format string is defined here 710 | "index of %ld is too large (%s has %ld params)", | ~~^ | | | long int | %d /wrkdirs/usr/ports/lang/gcc12-devel/work/gcc-12-20211121/gcc/jit/libgccjit.c: 184:35: error: format '%ld' expects argument of type 'long int', but argument 7 has type 'size_t' {aka 'unsigned int'} [-Werror=format=] My regular testers are x86-64 and do not show this, but if I'm right it should also should on 32-bit GNU/Linux? Gerald