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 62379162412A; Thu, 15 Jan 2026 11:04:44 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org 62379162412A DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=tarantool.org; s=dev; t=1768464284; bh=ijEjF8VTaQsVra173BLxUJVeC9L9xDmtpLulZMsKGUI=; h=Date:To:Cc:References:In-Reply-To:Subject:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From:Reply-To:From; b=PJ0VIpMt8CCOj5vEpWFGgu61CmILoNuSgas5zk/YXCaGXAF1ahOj9GWb2/wmrthzK 8DuhyVTc2s+3VW0+Lm7j6GL0zjZTJ4lEz/ENPUqiqZj2tRYzL31ECSFUl7U4YidMXn /0q4YTQaCxHbsHcUF3kK1emMtqPx1UJHIcoeXs/w= Received: from send195.i.mail.ru (send195.i.mail.ru [95.163.59.34]) (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 dev.tarantool.org (Postfix) with ESMTPS id 6C1CB162411B for ; Thu, 15 Jan 2026 11:04:43 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org 6C1CB162411B Received: by exim-smtp-55f9c6db88-6pfzx with esmtpa (envelope-from ) id 1vgILS-00000000F4c-2NmA; Thu, 15 Jan 2026 11:04:42 +0300 Date: Thu, 15 Jan 2026 11:04:54 +0300 To: Sergey Bronnikov Cc: tarantool-patches@dev.tarantool.org Message-ID: References: <12e9c9662886c6b00f442e6f08722e0c066f401d.1766738771.git.skaplun@tarantool.org> <634cfd44-da7d-487e-ba01-a568ed55e3ea@tarantool.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <634cfd44-da7d-487e-ba01-a568ed55e3ea@tarantool.org> X-Mailru-Src: smtp X-4EC0790: 10 X-7564579A: 646B95376F6C166E X-77F55803: 4F1203BC0FB41BD99B884C1B59F9AF80C8DE077655354E83D5784140AFD5E85D182A05F538085040C1FB7BA4A3E082003DE06ABAFEAF67050337C76405C8AFA319EC99DA27725702DB2FAAB0A8254537 X-7FA49CB5: FF5795518A3D127A4AD6D5ED66289B5278DA827A17800CE7F2EC3597058CFA6DEA1F7E6F0F101C67BD4B6F7A4D31EC0BCC500DACC3FED6E28638F802B75D45FF8AA50765F7900637FE9EFE935CD7C6AE8638F802B75D45FF914D58D5BE9E6BC1A93B80C6DEB9DEE97C6FB206A91F05B23CF11D00B1B384F82E070BE324C7D3C468B40F910C122C53F6B57BC7E64490618DEB871D839B73339E8FC8737B5C2249D082881546D93491CC7F00164DA146DAFE8445B8C89999729449624AB7ADAF37F6B57BC7E64490611E7FA7ABCAF51C92176DF2183F8FC7C0D9442B0B5983000E8941B15DA834481F9449624AB7ADAF372E808ACE2090B5E14AD6D5ED66289B5259CC434672EE63711DD303D21008E298D5E8D9A59859A8B6B372FE9A2E580EFC725E5C173C3A84C309A7649CC036878F35872C767BF85DA2F004C90652538430E4A6367B16DE6309 X-C1DE0DAB: 0D63561A33F958A5AD4B9FC8E9B0A8CF5002B1117B3ED696B2CA42FB5D37406EB48B7A7F94616420823CB91A9FED034534781492E4B8EEAD5DF1C2DF01CE7211BDAD6C7F3747799A X-C8649E89: 1C3962B70DF3F0AD73CAD6646DEDE1918E10F71CB4DF9F96AB70F9BE574AE9C625B6776AC983F447FC0B9F89525902EE6F57B2FD27647F25E66C117BDB76D65917C03B4BE441EAE98F5B9E53F46E8619683E6EA7FC87DA384AFC77E1F76661A7933D47ED6C9F44B1B8341EE9D5BE9A0AF67630DBD193E5BA5775D4C8E9895601F6DB023EC1ED5A528CD93680B12512CF4C41F94D744909CECFA6C6B0C050A61A8CAF69B82BA93681CD72808BE417F3B9E0E7457915DAA85F X-D57D3AED: 3ZO7eAau8CL7WIMRKs4sN3D3tLDjz0dLbV79QFUyzQ2Ujvy7cMT6pYYqY16iZVKkSc3dCLJ7zSJH7+u4VD18S7Vl4ZUrpaVfd2+vE6kuoey4m4VkSEu53w8ahmwBjZKM/YPHZyZHvz5uv+WouB9+ObcCpyrx6l7KImUglyhkEat/+ysWwi0gdhEs0JGjl6ggRWTy1haxBpVdbIX1nthFXMZebaIdHP2ghjoIc/363UZI6Kf1ptIMVVt0N+pdRHOZNyncORb1Cdg= X-Mailru-Sender: 689FA8AB762F7393DDD5FD59B456EAD2C8E66F6788A0D1356FF0F0C62E4A9EA0D87DE97941F32E99E49D44BB4BD9522A059A1ED8796F048DB274557F927329BE89D5A3BC2B10C37545BD1C3CC395C826B4A721A3011E896F X-Mras: Ok Subject: Re: [Tarantool-patches] [PATCH v2 luajit 18/41] perf: adjust nsieve-bit-fp in LuaJIT-benches 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: Sergey Kaplun via Tarantool-patches Reply-To: Sergey Kaplun Errors-To: tarantool-patches-bounces@dev.tarantool.org Sender: "Tarantool-patches" Hi, Sergey! Thanks for the review! Please consider my answer below. On 15.01.26, Sergey Bronnikov wrote: > Hi, Sergey, > > thanks for the patch! > > Sergey > > On 12/26/25 12:17, Sergey Kaplun wrote: > > > > > + checker = function(res) > > + if N == DEFAULT_N then > > + assert(res[0] == 2488465) > > + assert(res[1] == 1299069) > > + assert(res[2] == 679461) > > Minor: I would leave a comment about magic numbers or use variables with > self-explained names. >From the code and the comment in the header, it is clear that this is the number of prime numbers till `(2 ^ (12 - i)) * 10000`. > > Feel free to ignore. Ignoring. > > > + end > > + return true > > + end, > > +} > > + > > +bench:add(benchmark) > > +bench:run_and_report() -- Best regards, Sergey Kaplun