From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from [87.239.111.99] (localhost [127.0.0.1]) by dev.tarantool.org (Postfix) with ESMTP id E50B96FFA3; Wed, 31 Mar 2021 22:25:19 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org E50B96FFA3 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=tarantool.org; s=dev; t=1617218719; bh=PqtQZk+w9uI4d35iJCRpDgdR39pOhHrG64jVEhwr37c=; h=Date:To:References:In-Reply-To:Subject:List-Id:List-Unsubscribe: List-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:Cc: From; b=RDylCm8psbXbOBZmEhM5sSjF49DVZ8WjMltYgVUjS1mYRx5+hpdqwJ84n0PMwjRUt nxiZIoqbfYwYXCNGHjNApmnvbJRVVfVWQQ1/pxf9tLkdLlveV9KWSat5xZDtngl41N RCxHD+1ezHLqLH83igdMFCBbTT4HTZNIWvwVJWjw= Received: from smtpng2.m.smailru.net (smtpng2.m.smailru.net [94.100.179.3]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dev.tarantool.org (Postfix) with ESMTPS id 8A3C86FFA3 for ; Wed, 31 Mar 2021 22:24:15 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org 8A3C86FFA3 Received: by smtpng2.m.smailru.net with esmtpa (envelope-from ) id 1lRgRq-0002yG-IU; Wed, 31 Mar 2021 22:24:14 +0300 Date: Wed, 31 Mar 2021 22:24:05 +0300 To: Sergey Kaplun Message-ID: <20210331192405.GM29703@tarantool.org> References: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: X-Clacks-Overhead: GNU Terry Pratchett User-Agent: Mutt/1.10.1 (2018-07-13) X-7564579A: B8F34718100C35BD X-77F55803: 4F1203BC0FB41BD9ED7173E37F4E32947A0146560F8BA70927CAA5B950F38D9F182A05F538085040E0B0545008AC723F1E610F036AE9BDEE44B1B03AA3B2B6BA9FEE8B6D55B15E37 X-7FA49CB5: FF5795518A3D127A4AD6D5ED66289B5278DA827A17800CE7544B1CCE26E01C74EA1F7E6F0F101C67BD4B6F7A4D31EC0BCC500DACC3FED6E28638F802B75D45FF8AA50765F7900637B997C8222C70C3D98638F802B75D45FF914D58D5BE9E6BC131B5C99E7648C95CE99938B3FD79E1DFA60956B51FDE6286F4F57E3C246210C2A471835C12D1D9774AD6D5ED66289B5259CC434672EE6371117882F4460429724CE54428C33FAD30A8DF7F3B2552694AC26CFBAC0749D213D2E47CDBA5A9658378DA827A17800CE7328B01A8D746D8839FA2833FD35BB23DF004C906525384302BEBFE083D3B9BA71A620F70A64A45A98AA50765F790063735872C767BF85DA227C277FBC8AE2E8B38189D631E94611675ECD9A6C639B01B4E70A05D1297E1BBCB5012B2E24CD356 X-B7AD71C0: AC4F5C86D027EB782CDD5689AFBDA7A2AD77751E876CB595E8F7B195E1C978312F0A634159382C964EBAF99501702FC7 X-C1DE0DAB: 0D63561A33F958A553A6660FDF0462BECCC5BDC5FAA550FCA443EFBA1FAA6DE6D59269BC5F550898D99A6476B3ADF6B47008B74DF8BB9EF7333BD3B22AA88B938A852937E12ACA7502E6951B79FF9A3F410CA545F18667F91A7EA1CDA0B5A7A0 X-C8649E89: 4E36BF7865823D7055A7F0CF078B5EC49A30900B95165D3429538671E6527D3279F3131A7D7CE9C1A0D60A1E1055E1AE40F86D47BD1896E0465CF07C2921F59A1D7E09C32AA3244C67CF5E1151D341A859D875520E054FF9408A6A02710B7304927AC6DF5659F194 X-D57D3AED: 3ZO7eAau8CL7WIMRKs4sN3D3tLDjz0dLbV79QFUyzQ2Ujvy7cMT6pYYqY16iZVKkSc3dCLJ7zSJH7+u4VD18S7Vl4ZUrpaVfd2+vE6kuoey4m4VkSEu530nj6fImhcD4MUrOEAnl0W826KZ9Q+tr5ycPtXkTV4k65bRjmOUUP8cvGozZ33TWg5HZplvhhXbhDGzqmQDTd6OAevLeAnq3Ra9uf7zvY2zzsIhlcp/Y7m53TZgf2aB4JOg4gkr2biojetunDCtJ20KRygLJ4s5Vkw== X-Mailru-Sender: 689FA8AB762F73936BC43F508A0638226A6DE8D2A1D0878EEFE8B983ED7B7F7FA7C8D0F45F857DBFE9F1EFEE2F478337FB559BB5D741EB964C8C2C849690F8E70A04DAD6CC59E33667EA787935ED9F1B X-Mras: Ok Subject: Re: [Tarantool-patches] [PATCH v2 luajit 25/30] test: disable PUC Lua error test for syntax level X-BeenThere: tarantool-patches@dev.tarantool.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Igor Munkin via Tarantool-patches Reply-To: Igor Munkin Cc: tarantool-patches@dev.tarantool.org Errors-To: tarantool-patches-bounces@dev.tarantool.org Sender: "Tarantool-patches" Sergey, Thanks for the patch! LGTM, except the nits below. On 26.03.21, Sergey Kaplun wrote: > When LuaJIT is compiled with LUAJIT_ENABLE_GC64, LJ_MAX_SLOTS limit is > reached and error LJ_ERR_XSLOTS ("function or expression too complex") > is raised earlier, than LJ_MAX_XLEVEL limit is reached and error > LJ_ERR_XLEVELS ("chunk has too many syntax levels") is raised. > > This patch disabled test expected the LJ_ERR_XLEVEL error, but Typo: s/disabled test/disables the test/. > failing with the LJ_ERR_XSLOTS error. > > Part of tarantool/tarantool#5845 > Part of tarantool/tarantool#4473 > --- > test/PUC-Lua-5.1-tests/errors.lua | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/test/PUC-Lua-5.1-tests/errors.lua b/test/PUC-Lua-5.1-tests/errors.lua > index 028224c..328976e 100644 > --- a/test/PUC-Lua-5.1-tests/errors.lua > +++ b/test/PUC-Lua-5.1-tests/errors.lua > @@ -228,7 +228,13 @@ local function testrep (init, rep) > end > testrep("a=", "{") > testrep("a=", "(") > -testrep("", "a(") > +-- LuaJIT: When compiled with LUAJIT_ENABLE_GC64, LJ_MAX_SLOTS > +-- limit is reached and error LJ_ERR_XSLOTS ("function or > +-- expression too complex") is raised earlier, than LJ_MAX_XLEVEL > +-- limit is reached and error LJ_ERR_XLEVELS ("chunk has too many > +-- syntax levels") is raised. Side note: It would be nice if you also mention the root cause. AFAIU, LuaJIT frontend checks "virtual" stack size at the translation time, right? Therefore, when GC64 support is enabled every call needs twice more slots on the coroutine stack (since LJ_FR2 is also set). > +-- Test is disabled for LuaJIT. > +-- testrep("", "a(") > testrep("", "do ") > testrep("", "while a do ") > testrep("", "if a then else ") > -- > 2.31.0 > -- Best regards, IM