From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from esa3.mentor.iphmx.com (esa3.mentor.iphmx.com [68.232.137.180]) by sourceware.org (Postfix) with ESMTPS id 2EEDF3858421 for ; Thu, 8 Sep 2022 15:29:19 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 2EEDF3858421 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=codesourcery.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=mentor.com X-IronPort-AV: E=Sophos;i="5.93,300,1654588800"; d="diff'?scan'208";a="82550583" Received: from orw-gwy-01-in.mentorg.com ([192.94.38.165]) by esa3.mentor.iphmx.com with ESMTP; 08 Sep 2022 07:29:18 -0800 IronPort-SDR: 0YTgr5VWOXEoqAqyJlHQ8/Q747BI8vAVSkQE5dXeQiSmuD1hi9E+4R2gop75cvDt17eCJx8FSL Fl+AwJMmjjN0zLU/PHJaDbm+VTgrmG0aRVzO1/i3TIKnDmJUPGBB9qfE5icTRVlzqTsongliW9 7yKD4ANWLWMEVR5Z98PH6nRb3f+r7f0veCClQaB3XrfS8fxiCPuTPgzPj3tJkXQJMa8LVLmA9g FUFUSQxXkyHyqscunbuV7lPQxnpOtjffxLN7fzZ/qAKaZc5JnRG53W/wxLMSv3zGlcEAo9BgDi 0eo= Content-Type: multipart/mixed; boundary="------------1rEOE86nVYdDYuLDgPCE56q2" Message-ID: <56951572-c9b4-af2d-0e8b-9d47b87ba313@codesourcery.com> Date: Thu, 8 Sep 2022 23:29:12 +0800 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0) Gecko/20100101 Thunderbird/102.2.1 Content-Language: en-US From: Chung-Lin Tang To: gcc-patches , Joseph Myers CC: Sandra Loosemore , Jan-Benedict Glaw Subject: [PATCH] optc-save-gen.awk: adjust generated array compare X-ClientProxiedBy: SVR-ORW-MBX-07.mgc.mentorg.com (147.34.90.207) To svr-orw-mbx-10.mgc.mentorg.com (147.34.90.210) X-Spam-Status: No, score=-10.1 required=5.0 tests=BAYES_00,GIT_PATCH_0,HEADER_FROM_DIFFERENT_DOMAINS,KAM_ASCII_DIVIDERS,KAM_DMARC_STATUS,SPF_HELO_PASS,SPF_PASS,TXREP,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: --------------1rEOE86nVYdDYuLDgPCE56q2 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Hi Joseph, Jan-Benedict reported a build-bot error for the nios2 port under --enable-werror-always: options-save.cc: In function 'bool cl_target_option_eq(const cl_target_option*, const cl_target_option*)': options-save.cc:9291:38: error: comparison between two arrays [-Werror=array-compare] 9291 | if (ptr1->saved_custom_code_status != ptr2->saved_custom_code_status | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ options-save.cc:9291:38: note: use unary '+' which decays operands to pointers or '&'component_ref' not supported by dump_decl[0] != &'component_ref' not supported by dump_decl[0]' to compare the addresses options-save.cc:9294:37: error: comparison between two arrays [-Werror=array-compare] 9294 | if (ptr1->saved_custom_code_index != ptr2->saved_custom_code_index | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ... This is due to an array-typed TargetSave state in config/nios2/nios2.opt: ... TargetSave enum nios2_ccs_code saved_custom_code_status[256] TargetSave int saved_custom_code_index[256] ... This patch adjusts the generated array state compare from 'ptr1->array' into '&ptr1->array[0]' in gcc/optc-save-gen.awk, seems sufficient to pass the tougher checks. Tested by ensuring the compiler builds, which should be sufficient here. Okay to commit to mainline? Thanks, Chung-Lin * optc-save-gen.awk: Adjust array compare to use '&ptr->name[0]' instead of 'ptr->name'. --------------1rEOE86nVYdDYuLDgPCE56q2 Content-Type: text/plain; charset="UTF-8"; name="a.diff" Content-Disposition: attachment; filename="a.diff" Content-Transfer-Encoding: base64 ZGlmZiAtLWdpdCBhL2djYy9vcHRjLXNhdmUtZ2VuLmF3ayBiL2djYy9vcHRjLXNhdmUtZ2Vu LmF3awppbmRleCAyMzNkMWZiYjYzNy4uMjdhYWJmMjk1NWUgMTAwNjQ0Ci0tLSBhL2djYy9v cHRjLXNhdmUtZ2VuLmF3aworKysgYi9nY2Mvb3B0Yy1zYXZlLWdlbi5hd2sKQEAgLTEwOTMs NyArMTA5Myw3IEBAIGZvciAoaSA9IDA7IGkgPCBuX3RhcmdldF9hcnJheTsgaSsrKSB7CiAJ bmFtZSA9IHZhcl90YXJnZXRfYXJyYXlbaV0KIAlzaXplID0gdmFyX3RhcmdldF9hcnJheV9z aXplW2ldCiAJdHlwZSA9IHZhcl90YXJnZXRfYXJyYXlfdHlwZVtpXQotCXByaW50ICIgIGlm IChwdHIxLT4iIG5hbWUiICE9IHB0cjItPiIgbmFtZSAiIjsKKwlwcmludCAiICBpZiAoJnB0 cjEtPiIgbmFtZSJbMF0gIT0gJnB0cjItPiIgbmFtZSAiWzBdIjsKIAlwcmludCAiICAgICAg fHwgbWVtY21wIChwdHIxLT4iIG5hbWUgIiwgcHRyMi0+IiBuYW1lICIsICIgc2l6ZSAiICog c2l6ZW9mKCIgdHlwZSAiKSkpIgogCXByaW50ICIgICAgcmV0dXJuIGZhbHNlOyI7CiB9Cg== --------------1rEOE86nVYdDYuLDgPCE56q2--