From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by sourceware.org (Postfix) with ESMTPS id C8AF03858405 for ; Wed, 30 Mar 2022 12:19:25 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org C8AF03858405 Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-538-jT9nkjRkPXGgoFaVJVvMJQ-1; Wed, 30 Mar 2022 08:19:24 -0400 X-MC-Unique: jT9nkjRkPXGgoFaVJVvMJQ-1 Received: from smtp.corp.redhat.com (int-mx02.intmail.prod.int.rdu2.redhat.com [10.11.54.2]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id E6F078F11D6; Wed, 30 Mar 2022 12:19:23 +0000 (UTC) Received: from [10.97.116.51] (ovpn-116-51.gru2.redhat.com [10.97.116.51]) by smtp.corp.redhat.com (Postfix) with ESMTPS id BDDB0400E43D; Wed, 30 Mar 2022 12:19:22 +0000 (UTC) Message-ID: <9c0115ea-7178-91ee-9614-586d243b578c@redhat.com> Date: Wed, 30 Mar 2022 09:19:18 -0300 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.7.0 Subject: Re: [PATCH 09/11] gdb/testsuite: don't test gdb.base/msym-bp-shl with clang To: Pedro Alves , gdb-patches@sourceware.org References: <20220126195053.69559-1-blarsen@redhat.com> <20220126195053.69559-10-blarsen@redhat.com> From: Bruno Larsen In-Reply-To: X-Scanned-By: MIMEDefang 2.84 on 10.11.54.2 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Language: en-US Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-12.6 required=5.0 tests=BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, NICE_REPLY_A, RCVD_IN_DNSWL_LOW, RCVD_IN_MSPIKE_H5, RCVD_IN_MSPIKE_WL, SPF_HELO_NONE, SPF_NONE, TXREP, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 30 Mar 2022 12:19:27 -0000 On 3/2/22 16:33, Pedro Alves wrote: > On 2022-01-26 19:50, Bruno Larsen via Gdb-patches wrote: >> Clang will optimize away the static function in one of the files, and >> the test is here to specifically test GDB's behavior when that function >> is present, so it makes no sense to have this test run with that >> compiler. > > Please expand this info to include a snippet of the failing test. > Also, it's best if the the body of the commit log, doesn't assume the subject is read > as part of it. You will notice that if you read the body in isolation, without the > subject line, and also without the diff context, the commit log is vague as is. Will do! Sorry for the delay, got sidetracked with another issue. > >> --- >> gdb/testsuite/gdb.base/msym-bp-shl.exp | 7 +++++++ >> 1 file changed, 7 insertions(+) >> >> diff --git a/gdb/testsuite/gdb.base/msym-bp-shl.exp b/gdb/testsuite/gdb.base/msym-bp-shl.exp >> index 42adcb191dd..95d5c393505 100644 >> --- a/gdb/testsuite/gdb.base/msym-bp-shl.exp >> +++ b/gdb/testsuite/gdb.base/msym-bp-shl.exp >> @@ -22,6 +22,13 @@ if {[skip_shlib_tests]} { >> return 0 >> } >> >> +# clang will optimize away the static foo, making a single breakpoint >> +# so there is no point testing it here. > > Uppercase Clang. > > The patch doesn't give enough info to understand why this is the best approach. > I could imagine using atttribute used to make sure the function isn't optimized > away, as alternative approach, for example. Please make the case for the approach > taken. I tried using the attribute approach (after you mentioned it here), but clang still optimized the static foo away. I'll document it on the commit message and the comment here for v2. > >> +if {[test_compiler_info {clang-*-*}]} { >> + untested "clang only compiles one foo" >> + return >> +} >> + >> standard_testfile msym-bp-shl-main.c msym-bp-shl-main-2.c msym-bp-shl-lib.c >> set srcfile ${srcdir}/${subdir}/${srcfile} >> set srcfile2 ${srcdir}/${subdir}/${srcfile2} > -- Cheers! Bruno Larsen