From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 39379 invoked by alias); 13 Oct 2017 08:01:12 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 39339 invoked by uid 89); 13 Oct 2017 08:01:10 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.9 required=5.0 tests=BAYES_00,SPF_PASS autolearn=ham version=3.3.2 spammy=HContent-Transfer-Encoding:8bit X-HELO: mx2.suse.de Received: from mx2.suse.de (HELO mx2.suse.de) (195.135.220.15) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Fri, 13 Oct 2017 08:01:04 +0000 Received: from relay2.suse.de (charybdis-ext.suse.de [195.135.220.254]) by mx2.suse.de (Postfix) with ESMTP id 44056AAEF; Fri, 13 Oct 2017 08:01:02 +0000 (UTC) Subject: Re: [PATCH] Fix bitmap_bit_in_range_p (PR tree-optimization/82493). To: Jeff Law , gcc-patches@gcc.gnu.org References: <9c9fd60f-cb7a-e702-aabb-9e31dca6a92a@suse.cz> <2215478f-6715-189a-e6a4-8d171901d31f@redhat.com> From: =?UTF-8?Q?Martin_Li=c5=a1ka?= Message-ID: <467ec34e-7af8-b5b8-264a-a336bd810e49@suse.cz> Date: Fri, 13 Oct 2017 08:01:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 MIME-Version: 1.0 In-Reply-To: <2215478f-6715-189a-e6a4-8d171901d31f@redhat.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-IsSubscribed: yes X-SW-Source: 2017-10/txt/msg00833.txt.bz2 On 10/12/2017 11:54 PM, Jeff Law wrote: > On 10/11/2017 12:13 AM, Martin Liška wrote: >> 2017-10-10 Martin Liska >> >> PR tree-optimization/82493 >> * sbitmap.c (bitmap_bit_in_range_p): Fix the implementation. >> (test_range_functions): New function. >> (sbitmap_c_tests): Likewise. >> * selftest-run-tests.c (selftest::run_tests): Run new tests. >> * selftest.h (sbitmap_c_tests): New function. > I went ahead and committed this along with a patch to fix the off-by-one > error in live_bytes_read. Bootstrapped and regression tested on x86. > > Actual patch attached for archival purposes. > > Jeff > Thanks. I'll carry on and write another set of unit tests. Martin