From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by sourceware.org (Postfix) with ESMTPS id 77BAD3858D1E for ; Tue, 20 Jun 2023 14:17:52 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 77BAD3858D1E Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=intel.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=intel.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1687270672; x=1718806672; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-transfer-encoding:mime-version; bh=/gF9cGgn1E6eofeIRRScTTjrOyDyq+8gC6IMdKV8NMk=; b=QxBTsVBsiN166iJUJgs9V9kUQjaPp9S4aZHZdjUPawMT1eha1Sg1M2ne 47WZ6LU3LmW/dJHFl4riUfoXw0254k9//XP/DfNmpsk3ggw8rrB3iMaTB cU3GSZ2x62jhZiHeyS9MzQvaL1szq28QmVVFqIn9hEAU4J+04AJdJfx4Y jrYRZQFW2yKwGb+tSL+hx6x3L2rfPMDmJz5x//On70Z1Ytb6FXgTFg5/u c2YJZy0pIRBmW3j05gjIlcEs/GWl7V3R8/yeDe1UeGodVO0fAziq/YXso DwV2uXUoSen6CUl4hn1eJDnxYRNcqRZG8x3cB0cYHquLvbqawGRXkBaK9 Q==; X-IronPort-AV: E=McAfee;i="6600,9927,10747"; a="389126505" X-IronPort-AV: E=Sophos;i="6.00,257,1681196400"; d="scan'208";a="389126505" Received: from orsmga007.jf.intel.com ([10.7.209.58]) by fmsmga101.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 20 Jun 2023 07:08:13 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10747"; a="708292311" X-IronPort-AV: E=Sophos;i="6.00,257,1681196400"; d="scan'208";a="708292311" Received: from fmsmsx603.amr.corp.intel.com ([10.18.126.83]) by orsmga007.jf.intel.com with ESMTP; 20 Jun 2023 07:08:12 -0700 Received: from fmsmsx611.amr.corp.intel.com (10.18.126.91) by fmsmsx603.amr.corp.intel.com (10.18.126.83) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.23; Tue, 20 Jun 2023 07:08:12 -0700 Received: from fmsmsx612.amr.corp.intel.com (10.18.126.92) by fmsmsx611.amr.corp.intel.com (10.18.126.91) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.23; Tue, 20 Jun 2023 07:08:11 -0700 Received: from fmsedg601.ED.cps.intel.com (10.1.192.135) by fmsmsx612.amr.corp.intel.com (10.18.126.92) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.23 via Frontend Transport; Tue, 20 Jun 2023 07:08:11 -0700 Received: from NAM10-DM6-obe.outbound.protection.outlook.com (104.47.58.105) by edgegateway.intel.com (192.55.55.70) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.23; Tue, 20 Jun 2023 07:08:11 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Jfo6PrmUqZUnb6cXRV67AKwmRBShD+fIPcweXI0qUQmPzK8BCv5N1okXaRrtClpZLlpulGze4VSOKwWEaiP8vdQfEx+iB9JyTX1yg9FMKLBiMaX3bapPg8U4NddlVtZxGtoE/Chk4rzJbxJ1BHp3vlfkZaRaaqcT97anNnIAPzaLOAttOBq4XPGZQHNHJ+sDS2/pLh+TSb7CC6BTRrPctGL7Zwq49YYimvkI/naSpNnx457jowxUuhGc6ss/ZQiyI+5y5gVut+MK+W3PbeN24PcJSB6IOYj6FOkiH7ElYFnTUEa7j79A/OafM3lO8KWwLj0QQZ04q1vn3jnIIBTFEg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=M7zv0jL3hsdnSRqKdPzmtXPcopzwZsC+6Jq+JeHeskU=; b=diruCxPkyyLMnX2z3y4jex/Ab+iGGJxchJvsYkIZ0u1fSj0HsO0Uif1HNBIZDUQBwFej9gwXwHgBLwUZHeeRLSgBFfhUYqCWxVhV24Iw+dmN6OnQhNNJoyK2GncmPEMyJ+LdJCCK/7J+xJaOu+RQw2ha619uAxcGMyVS3QkS0vNXImw6oHBb+1/jKlN9e+viMWqkU4oweIV2OCd0OPJcTcqvIvFIN8PN2OefCMQv5NlRT6jMuaboQt/g31sQAoDsEmOdC7l6vWYSCuZzHenvfXgy1PGoTXG1lZb6DvXR7wX57IAAofK3yxqajO4ixLZ1yLi1o/FxU25MedG8Osktvw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none Received: from MW5PR11MB5908.namprd11.prod.outlook.com (2603:10b6:303:194::10) by CH0PR11MB5282.namprd11.prod.outlook.com (2603:10b6:610:bd::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6500.37; Tue, 20 Jun 2023 14:08:08 +0000 Received: from MW5PR11MB5908.namprd11.prod.outlook.com ([fe80::127c:f4cc:e699:8f73]) by MW5PR11MB5908.namprd11.prod.outlook.com ([fe80::127c:f4cc:e699:8f73%6]) with mapi id 15.20.6500.036; Tue, 20 Jun 2023 14:08:07 +0000 From: "Li, Pan2" To: Jakub Jelinek CC: "gcc-patches@gcc.gnu.org" , "juzhe.zhong@rivai.ai" , "rdapp.gcc@gmail.com" , "jeffreyalaw@gmail.com" , "Wang, Yanzhang" , "kito.cheng@gmail.com" , "rguenther@suse.de" Subject: RE: [PATCH] RISC-V: Fix out of range memory access of machine mode table Thread-Topic: [PATCH] RISC-V: Fix out of range memory access of machine mode table Thread-Index: AQHZoo1G5H0Lv2r9XkuUWLbdvLdQGK+R2HQAgABIDhCAARQ0EIAAIaEAgABlIPA= Date: Tue, 20 Jun 2023 14:08:07 +0000 Message-ID: References: <20230619080710.1536456-1-pan2.li@intel.com> <20230619090548.1574008-1-pan2.li@intel.com> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=intel.com; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: MW5PR11MB5908:EE_|CH0PR11MB5282:EE_ x-ms-office365-filtering-correlation-id: c82aa7d9-dee5-4f02-46a8-08db7197c5a9 x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: InhhI9yCi2C84SkKXdgbGE6ktlkmUqiPtn9saOsVakKyKENzz/elhx0Dqne8y9v0KaW1JpSRYe9WgtatI/jjR1rPH3ZRQGg/Ije385wxcIl/vFhnOZChNhhML+D70EPy7rSSoH+uhPIbBXeRgNlAFyrbJAc6JcQ2WijxoBPG/fnRgOrN9O6J2UgX6jxP1ute8pp87TULtIEjjvAsdiEbuulO878/N16rQU/iAEilTpPk4cix9DhKBhxB6/onD5z+uQ+gJD5Y+hvaNqFbX5O4tRXuodg50O2Tt+I4SyhGXMAtuJ+s3Bfn4fuZLDBRl5GnXGSLigpHlOJ95QmhKR9ubw4rIZRl7pp2GmWIBGWHtL2xHH8qecTEfw/b/oZmrLFNl12aioNOpOXnlF82w1oYsh6BPEUXozoBxlf18Qp0hULq4I/II2eoAcoCkaPGhdjdz7PWH3C3y94If6EkY+5rizT7Kl3lVhjEi8nDxuooW4AmS9Q5nbFYFZiecRMTa+YD+5yCIW84aeMV4eZprZCX89U6AttG6R8sMD67BnTDe75aNROUv+la8hhXzDp2NJ/xCq0NDG4lqB5JV+PARi18Crx6k6VonrTXF7AJxLOlQbm94woLcDi5EL6SBs7ncKzc x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:MW5PR11MB5908.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230028)(376002)(346002)(366004)(39860400002)(396003)(136003)(451199021)(478600001)(7696005)(71200400001)(83380400001)(54906003)(6506007)(53546011)(26005)(186003)(9686003)(2906002)(5660300002)(52536014)(33656002)(38100700002)(122000001)(76116006)(66946007)(55016003)(82960400001)(4326008)(316002)(8936002)(8676002)(41300700001)(86362001)(66476007)(6916009)(64756008)(66556008)(66446008)(38070700005);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?k96d7AmvU6K8UrhbPt91TQuY5FM1vdf0D7cXxlKw+5GY5fmsYqjHpB5RZBM1?= =?us-ascii?Q?o17Gnvw02fOl3NT5NLGx7kaWVXDWNPFv3RMhDtg3MhyVSdmhPANMt2wSQ988?= =?us-ascii?Q?6yB4dOy1+Pbd59SIilqfITJIDAiqTx+g9YyamCApDhay+tlnLJaud9gOydfw?= =?us-ascii?Q?YM2gSn/4G8vVOn9VuBBCY2q9x85GW5ZkGqTr0e9urq2GmOACD41SZepEko+F?= =?us-ascii?Q?Sa/+aDAM+qtkcEs++k/I0mAAGUnvPk+Ocp4ObnUg2KeDeShOViHUVytEz84F?= =?us-ascii?Q?vRD+wrOAEFrZqGYYfvsGfTa7gwbwcCVdMKUtUTO+0AeIqAN+zO2gkMkZeigP?= =?us-ascii?Q?LBzv/9NfvjtbaJEOHkFDslc742obGtaH3XMD4+3ErAEK6fJ4gg1wd+sxGTGy?= =?us-ascii?Q?KjsC3YvSasOfbINfExsUy2FrHMhcwDzBPaQUJeypQuS3B+1H+u7TXozRu3sx?= =?us-ascii?Q?Z0HjYNSEYMojLAW1uX5Fn2pVW5PQyfokkdh195mhukzJh69Tqm/tINZR9zRd?= =?us-ascii?Q?hwbZzVco+yk5uGkEQzt0BmoCh/ElGc9cxK9lqxtSufkDmTNm6np6/hZ95XPN?= =?us-ascii?Q?c3cMZQrJxyRgXqGk0xGDuPlp7dmX9rzm1RLjMTtucxVFJ7pcK6l/IMIDuC6J?= =?us-ascii?Q?uv3Fmpjw9eF7WK0K20HIyd7yPxmEmbG4ULrHQuyHTkP3nHt2GOB7bZD30/Xx?= =?us-ascii?Q?6kg1QXmcQA54wB51W7mJNSvLNSb7m8JCLlEolCJZueQAsjp0WiVr45+JH6Ua?= =?us-ascii?Q?0zx9gqd5APsGnTSRFt8Hgr+ZstnRNMzlhpRjsxCwJH1WquazpK0yX9LvVerH?= =?us-ascii?Q?hPEcKE2H2guEubFkwCkQAyBy6PBSDqnXujms5nubEm5t0SIOzjwSD+NIN0hN?= =?us-ascii?Q?0OQgal1JAereZ02i3cpgTjwaP+4SbtB7UNAncUIJPL8FmfPmBIg9IiyNNgDs?= =?us-ascii?Q?GI2tDibmOFHn0sxvwzvmy/zgdFN49Nw1J/frizj66hJiA6hrWeU86TDnSy0n?= =?us-ascii?Q?Vv+JOCCjf+KqkJerb1QPyovH9Bgf/r50TYKYk4Lz0/87fbafdT+BGc222Lyz?= =?us-ascii?Q?b2M5CWPQMB+u2zw8iFZ7qL3/GTJl+xLF3XX/1cbau1hIVYyAJ0ecvZoStPik?= =?us-ascii?Q?C4m1N6WOPJnTbLqbCe0TAeEK6oJCwUIsWeTONWEM87CULF0L3EToZAhrhvpy?= =?us-ascii?Q?YbOtm0Qzoj5DeIlsnybkSzUjc6LlGI9SM7cw/11bYvCpf00OE8i3sEiVC/nJ?= =?us-ascii?Q?U4woqo9Z1W6J5W410xNdFUeShbcZ0QEb3DU3gH4/kz8EkQw38HT53fNQ0158?= =?us-ascii?Q?jlFTLlK25emx7L3tWWHVUXmIqMg/KMXkd41MDsQfJ6P7LRXZwg3V77j0x2QC?= =?us-ascii?Q?XFiDD+BKh/D5lLb5RzpjcAYRRPrHhS5vqSxOZ/JoIo5VkDSR7Yux/18elXu5?= =?us-ascii?Q?dFHXNVlp0O86Bkddhd/M98WDAAa+hjGBQAUOW1fpVTaq3HQUyfzb/deOvsSM?= =?us-ascii?Q?LA/S2N6MF/lLVLpc7TuMXK8nQCGD2hynYCY3U3KeVpsCR22DGvUkMd6fLkmv?= =?us-ascii?Q?iOYUdrQFS6M7sEzsyWA=3D?= Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: MW5PR11MB5908.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: c82aa7d9-dee5-4f02-46a8-08db7197c5a9 X-MS-Exchange-CrossTenant-originalarrivaltime: 20 Jun 2023 14:08:07.2712 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: WrjGq+an9A5Z/c8s08l7+jg6cD00Hu0CKAa7zVJDAjRxWei6XfY8IFdOhQHN9N7i2yJT3n0fPW4Cfa5ZZD5/xw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH0PR11MB5282 X-OriginatorOrg: intel.com X-Spam-Status: No, score=-12.0 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,GIT_PATCH_0,RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,SPF_NONE,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: Thanks Jakub for the explanation, I have a try like below patch but I am no= t quite sure it is expected, and where should I put the assertion. > If yes, it needs to > be unsigned short, if not, we should add an assertion (e.g. on streaming > in the LTO table) that MAX_MACHINE_MODE <=3D 256. diff --git a/gcc/lto-streamer-in.cc b/gcc/lto-streamer-in.cc index 2cb83406db5..93ef97ec5d3 100644 --- a/gcc/lto-streamer-in.cc +++ b/gcc/lto-streamer-in.cc @@ -1985,8 +1985,6 @@ lto_input_mode_table (struct lto_file_decl_data *file= _data) internal_error ("cannot read LTO mode table from %s", file_data->file_name); =20 - unsigned char *table =3D ggc_cleared_vec_alloc (1 << 8); - file_data->mode_table =3D table; const struct lto_simple_header_with_strings *header =3D (const struct lto_simple_header_with_strings *) data; int string_offset; @@ -1994,6 +1992,9 @@ lto_input_mode_table (struct lto_file_decl_data *file= _data) string_offset =3D sizeof (*header) + header->main_size; =20 lto_input_block ib (data + sizeof (*header), header->main_size, NULL); + unsigned char *table =3D ggc_cleared_vec_alloc ( + 1 << ib.mode_bits); + file_data->mode_table =3D table; data_in =3D lto_data_in_create (file_data, data + string_offset, header->string_size, vNULL); bitpack_d bp =3D streamer_read_bitpack (&ib); @@ -2001,13 +2002,13 @@ lto_input_mode_table (struct lto_file_decl_data *fi= le_data) table[VOIDmode] =3D VOIDmode; table[BLKmode] =3D BLKmode; unsigned int m; - while ((m =3D bp_unpack_value (&bp, 8)) !=3D VOIDmode) + while ((m =3D bp_unpack_value (&bp, ib.mode_bits)) !=3D VOIDmode) { enum mode_class mclass =3D bp_unpack_enum (&bp, mode_class, MAX_MODE_CLASS); poly_uint16 size =3D bp_unpack_poly_value (&bp, 16); poly_uint16 prec =3D bp_unpack_poly_value (&bp, 16); - machine_mode inner =3D (machine_mode) bp_unpack_value (&bp, 8); + machine_mode inner =3D (machine_mode) bp_unpack_value (&bp, ib.mode_= bits); poly_uint16 nunits =3D bp_unpack_poly_value (&bp, 16); unsigned int ibit =3D 0, fbit =3D 0; unsigned int real_fmt_len =3D 0; @@ -2018,8 +2019,8 @@ lto_input_mode_table (struct lto_file_decl_data *file= _data) case MODE_UFRACT: case MODE_ACCUM: case MODE_UACCUM: - ibit =3D bp_unpack_value (&bp, 8); - fbit =3D bp_unpack_value (&bp, 8); + ibit =3D bp_unpack_value (&bp, ib.mode_bits); + fbit =3D bp_unpack_value (&bp, ib.mode_bits); break; case MODE_FLOAT: case MODE_DECIMAL_FLOAT: diff --git a/gcc/lto-streamer.h b/gcc/lto-streamer.h index fc7133d07ba..f1d826d59e4 100644 --- a/gcc/lto-streamer.h +++ b/gcc/lto-streamer.h @@ -352,6 +352,8 @@ public: =20 const char *data; const unsigned char *mode_table; + /* Indicates how many bits of one machine mode will have. */ + const unsigned int mode_bits =3D ceil_log2 (MAX_MACHINE_MODE) ; unsigned int p; unsigned int len; }; diff --git a/gcc/tree-streamer.cc b/gcc/tree-streamer.cc index ed65a7692e3..a28ef9c7920 100644 --- a/gcc/tree-streamer.cc +++ b/gcc/tree-streamer.cc @@ -35,7 +35,7 @@ along with GCC; see the file COPYING3. If not see During streaming in, we translate the on the disk mode using this table. For normal LTO it is set to identity, for ACCEL_COMPILER depending on the mode_table content. */ -unsigned char streamer_mode_table[1 << 8]; +unsigned char streamer_mode_table[MAX_MACHINE_MODE]; =20 /* Check that all the TS_* structures handled by the streamer_write_* and streamer_read_* routines are exactly ALL the structures defined in diff --git a/gcc/tree-streamer.h b/gcc/tree-streamer.h index 170d61cf20b..9aa248cd2f5 100644 --- a/gcc/tree-streamer.h +++ b/gcc/tree-streamer.h @@ -75,7 +75,7 @@ void streamer_write_tree_body (struct output_block *, tre= e); void streamer_write_integer_cst (struct output_block *, tree); =20 /* In tree-streamer.cc. */ -extern unsigned char streamer_mode_table[1 << 8]; +extern unsigned char streamer_mode_table[MAX_MACHINE_MODE]; void streamer_check_handled_ts_structures (void); bool streamer_tree_cache_insert (struct streamer_tree_cache_d *, tree, hashval_t, unsigned *); @@ -108,15 +108,18 @@ inline void bp_pack_machine_mode (struct bitpack_d *bp, machine_mode mode) { streamer_mode_table[mode] =3D 1; - bp_pack_enum (bp, machine_mode, 1 << 8, mode); + int last =3D 1 << ceil_log2 (MAX_MACHINE_MODE); + + bp_pack_enum (bp, machine_mode, last, mode); } =20 inline machine_mode bp_unpack_machine_mode (struct bitpack_d *bp) { - return (machine_mode) - ((class lto_input_block *) - bp->stream)->mode_table[bp_unpack_enum (bp, machine_mode, 1 << 8)]; + lto_input_block *input_block =3D (class lto_input_block *)bp->stream; + int index =3D bp_unpack_enum (bp, machine_mode, input_block->mode_bits); + + return (machine_mode)input_block->mode_table[index]; } =20 #endif /* GCC_TREE_STREAMER_H */ Pan -----Original Message----- From: Jakub Jelinek =20 Sent: Tuesday, June 20, 2023 4:04 PM To: Li, Pan2 Cc: gcc-patches@gcc.gnu.org; juzhe.zhong@rivai.ai; rdapp.gcc@gmail.com; jef= freyalaw@gmail.com; Wang, Yanzhang ; kito.cheng@gm= ail.com; rguenther@suse.de Subject: Re: [PATCH] RISC-V: Fix out of range memory access of machine mode= table On Tue, Jun 20, 2023 at 07:50:00AM +0000, Li, Pan2 wrote: > Hi Jakub, >=20 > Thanks for reviewing but I am not quite sure if I fully understand how to= fix this issue. Could you please help to enlighten me more about this ? >=20 > Currently for RISC-V, the memset has touched out of range memory already = due to MAX_MACHINE_MODE > 256. And we may have below parts require adjustin= g. >=20 > 1. streamer_mode_table. > 2. bp_unpack_machine_mode/bp_pack_machine_mode=20 > 3. bp_pack_value/bp_unpack_value in lto_write_mode_table. > 4. unsigned char *table =3D ggc_cleared_vec_alloc (1 << 8)= in lto_input_mode_table. >=20 > For 1. is safe to extend the size to MAX_MACHINE_MODE as the array only u= sed as Boolean, aka streamer_mode_table[XXXmode] =3D 1. Because the array is used only during stream out, that is safe. > For 2 & 3. Keep 1 << 8 as is, or stream out the host MAX_MACHINE_MODE val= ue somewhere for underlying consuming? You can't keep 1 << 8, otherwise you won't stream all the bits. I think you want to use 1 << ceil_log2 (MAX_MACHINE_MODE) on the stream out side, stream that ceil_log2 (MAX_MACHINE_MODE) value somewhere at the start of the mode table, add some field next to mode_table in lto_input_block which will contain that value (and make sure to initialize it to ceil_log2 (MAX_MACHINE_MODE) in case mode table isn't streamed in and use 1 << ...->mode_bits in e.g. bp_unpack_machine_mode Or for cases where 8 was used before use ceil_log2 (MAX_MACHINE_MODE) or mode_bits. > For 4, one possible approach is that extend unsigned char to unsigned sho= rt, as well as 256 to MAX_MACHINE_MODE. Because it stores the actually mach= ine mode in array. The 1 << 8 needs to be similarly 1 << ...->mode_bits or ...->num_modes (tha= t is also streamed out and in), it is sized by the host number of modes. Whether it is unsigned char or unsigned short array depends on if we want to support offloading targets with > 256 modes. If yes, it needs to be unsigned short, if not, we should add an assertion (e.g. on streaming in the LTO table) that MAX_MACHINE_MODE <=3D 256. Jakub