From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: AS4713 221.184.0.0/13 X-Spam-Status: No, score=-3.7 required=3.0 tests=AWL,BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,RCVD_IN_DNSWL_MED, SPF_HELO_NONE,SPF_PASS,UNPARSEABLE_RELAY shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2 Received: from neon.ruby-lang.org (neon.ruby-lang.org [221.186.184.75]) by dcvr.yhbt.net (Postfix) with ESMTP id F1FA31F4B4 for ; Fri, 9 Apr 2021 03:38:43 +0000 (UTC) Received: from neon.ruby-lang.org (localhost [IPv6:::1]) by neon.ruby-lang.org (Postfix) with ESMTP id BFEC6120B6B; Fri, 9 Apr 2021 12:37:38 +0900 (JST) Received: from xtrwkhkc.outbound-mail.sendgrid.net (xtrwkhkc.outbound-mail.sendgrid.net [167.89.16.28]) by neon.ruby-lang.org (Postfix) with ESMTPS id B0BB6120B6A for ; Fri, 9 Apr 2021 12:37:35 +0900 (JST) Received: by filterdrecv-59db977c98-mlp7z with SMTP id filterdrecv-59db977c98-mlp7z-14-606FCC39-31 2021-04-09 03:38:34.013787589 +0000 UTC m=+1247673.212446435 Received: from herokuapp.com (unknown) by geopod-ismtpd-canary-0 (SG) with ESMTP id pONHiHiJTBuATojTcDiUYw for ; Fri, 09 Apr 2021 03:38:33.840 +0000 (UTC) Date: Fri, 09 Apr 2021 03:38:34 +0000 (UTC) From: lamont@scriptkiddie.org Message-ID: References: Mime-Version: 1.0 X-Redmine-MailingListIntegration-Message-Ids: 79375 X-Redmine-Project: ruby-master X-Redmine-Issue-Tracker: Bug X-Redmine-Issue-Id: 17787 X-Redmine-Issue-Author: lamont X-Redmine-Sender: lamont X-Mailer: Redmine X-Redmine-Host: bugs.ruby-lang.org X-Redmine-Site: Ruby Issue Tracking System X-Auto-Response-Suppress: All Auto-Submitted: auto-generated X-SG-EID: =?us-ascii?Q?4Kn0dANS2V8yOnn1ZPtv+=2FZkTLjL6a8xnPqkRegpFALldJWLohvgct7XN4ScTv?= =?us-ascii?Q?avMNgZvmzv2Hdz2IX3lIlb7kBp+eibea49Hk0mu?= =?us-ascii?Q?TVlEc1qddYim=2FhXIicq7n3Yrqz4zaZRr2oeI4p8?= =?us-ascii?Q?0QvAsDjXakwCKknmlaae5IMWdAIyV3CRCVOqwx2?= =?us-ascii?Q?5jYLZvki5TSHFe4BPQJ3I2=2FEgn1mKm08U8RJqMR?= =?us-ascii?Q?6waVEttF++pJFyaPA=3D?= To: ruby-core@ruby-lang.org X-Entity-ID: b/2+PoftWZ6GuOu3b0IycA== X-ML-Name: ruby-core X-Mail-Count: 103329 Subject: [ruby-core:103329] [Ruby master Bug#17787] Four AIX build issues with xlc compiler and ruby-3.0.1 X-BeenThere: ruby-core@ruby-lang.org X-Mailman-Version: 2.1.15 Precedence: list Reply-To: Ruby developers List-Id: Ruby developers List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ruby-core-bounces@ruby-lang.org Sender: "ruby-core" Issue #17787 has been updated by lamont (Lamont Granquist). fdeclspec section is: ``` configure:9228: result: yes configure:9348: checking whether -fdeclspec is accepted as CFLAGS configure:9375: xlc_r -q64 -c -q64 -I/opt/chef/embedded/include -D_LARGE_FILES -O -fdeclspec -q64 -I/opt/chef/embedded/include -D_LARGE_FILES -O conftest.c >&5 configure:9375: $? = 0 configure:9400: result: yes configure:9425: checking whether -fdeclspec is accepted as CXXFLAGS configure:9457: xlC_r -q64 -o conftest -q64 -I/opt/chef/embedded/include -D_LARGE_FILES -O -fdeclspec -q64 -I/opt/chef/embedded/include -D_LARGE_FILES -O -q64 -L/opt/chef/embedded/lib -Wl,-blibpath:/opt/chef/e mbedded/lib:/usr/lib:/lib conftest.cpp >&5 ld: 0706-005 Cannot find or open file: declspec ld:fopen(): No such file or directory ``` later this trainwrecks the configure script with: ``` configure:14675: checking size of long long configure:14680: xlc_r -q64 -fdeclspec -o conftest -q64 -I/opt/chef/embedded/include -D_LARGE_FILES -O -q64 -I/opt/chef/embedded/include -D_LARGE_FILES -O -q64 -L/opt/chef/embedded/lib -Wl,-blibpath:/opt/chef/embedded/lib:/usr/lib:/lib conftest.c -lm >&5 ld: 0706-005 Cannot find or open file: declspec ld:fopen(): No such file or directory configure:14680: $? = 255 configure: program exited with status 255 ``` ---------------------------------------- Bug #17787: Four AIX build issues with xlc compiler and ruby-3.0.1 https://bugs.ruby-lang.org/issues/17787#change-91420 * Author: lamont (Lamont Granquist) * Status: Feedback * Priority: Normal * ruby -v: ruby 3.0.1p64 (2021-04-05 revision 0fb782ee38) [powerpc-aix7.1.5.0] * Backport: 2.5: UNKNOWN, 2.6: UNKNOWN, 2.7: UNKNOWN, 3.0: UNKNOWN ---------------------------------------- Yesterday found 4 small issues trying to use xlc to build ruby 3.0 on AIX that prevented building. My solutions are ugly fixes just to get our own builds going which will not be acceptable, but I'll include links to them. #1: internal/bits.h uses the gcc extension `__builtin_unreachable()` directly, this does not exist on xlc, I think it should be replaced by "`UNREACHABLE`" but I just used an empty statement: https://github.com/chef/omnibus-software/blob/3120a25111a29503a10866cbd4ee9fedcb438cee/config/patches/ruby/ruby-3.0.1-aix-builtin-unreachable.patch the ./configure script is correctly finding `rb_cv_func___builtin_unreachable=no` #2: With xlc to get cpp output you must use '`>`' and not '`-o`' which differs from `COUTFLAG` and this line in Makefile.in is incorrect and really needs better detection of this feature and not just assuming `CPPOUTFLAG = COUTFLAG`. I simply hardcoded the right value of `CPPOUTFLAG`: https://github.com/chef/omnibus-software/blob/3120a25111a29503a10866cbd4ee9fedcb438cee/config/patches/ruby/ruby-3.0.1-aix-cppoutflag.patch #3: The xlc compiler actually supports gcc's sync builtins. See the docs here: https://www.ibm.com/docs/en/SSGH3R_16.1.0/com.ibm.compilers.aix.doc/compiler.pdf#BIFS_GCC_ATOMIC The checks for this in the configure.ac script are guarded by a check for intel architecture which would need to be properly expanded properly in some way so that those checks would run on aix+xlc as well. I just forced it on: https://github.com/chef/omnibus-software/blob/3120a25111a29503a10866cbd4ee9fedcb438cee/config/patches/ruby/ruby-3.0.1-aix-force-gcc-sync-builtins.patch #4: The configure script incorrectly identifies the compiler as supporting `-fdeclspec`. It might be good to just omit these checks on the xlc compiler. I also tested the patches to `fdeclspec` which are currently unreleased in master since we needed to backport those to 3.0.1 for M1 mac builds. This one I just removed the `fdeclspec` checks entirely: https://github.com/chef/omnibus-software/blob/3120a25111a29503a10866cbd4ee9fedcb438cee/config/patches/ruby/ruby-3.0.1-aix-nofdeclspec.patch -- https://bugs.ruby-lang.org/