From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 14625 invoked by alias); 5 Sep 2013 20:52:38 -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 14616 invoked by uid 89); 5 Sep 2013 20:52:38 -0000 Received: from mail-qe0-f52.google.com (HELO mail-qe0-f52.google.com) (209.85.128.52) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-SHA encrypted) ESMTPS; Thu, 05 Sep 2013 20:52:38 +0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.7 required=5.0 tests=AWL,BAYES_00,KHOP_THREADED,NO_RELAYS autolearn=ham version=3.3.2 X-HELO: mail-qe0-f52.google.com Received: by mail-qe0-f52.google.com with SMTP id i11so1260762qej.39 for ; Thu, 05 Sep 2013 13:52:35 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:content-type; bh=Mn29GQr3SISdhP6iqs02m8kyLIcKTDifumEwmRGuPNk=; b=P4ODAvY25YHK3/2KfAjRrk+ugmbzxl5SwYL5FLFKfs+TZTiBfSzf9AvAQThuSnmPkJ h7JSMtFrXoGklUmirIglWN9By50XWe6PgHQr9CW0nY/gz422Hma2axluT9NbISvwN1CO jiXwl2N4Ssgjra0znhtPSDKcXpQS1ge3ShLzR1wT8Xuh9zUbZLP5zNPD2toN/atV/YKx 7w2NDKMggV0I8wu5nl0n2Jd2fg2mCzp7iwsFOozM82cb20Rezg3OrXsYDtB2NCf6Y+tz EZFrXxYqjX+BBFiyhuXO2qE18rIQZ6ve+QO2+WAID3roLk1BIXpr6evyf3TyFAxEhzOz 8C0w== X-Gm-Message-State: ALoCoQlQIY1eexiDa0zzecChdzki5QCo2hGTuDO9nk4LU+k7uLKlrcCknRDsw9LP5BJD6AK9mAhlo+jLTD86/AAf8gaKwYG3tZLLUK+30uAJgs1k3jG6sjX470LpqWlLY1byoBcb8fVPcUmvqqNIy+ZqmBVzEfRpXksJk+brfdiauu2QWsGKTjlwWKH6QOPulGzsumYmnu51oxSWl8M6tbMhJwZmIYwfeg== X-Received: by 10.49.131.200 with SMTP id oo8mr12579451qeb.0.1378414355215; Thu, 05 Sep 2013 13:52:35 -0700 (PDT) MIME-Version: 1.0 Received: by 10.229.46.72 with HTTP; Thu, 5 Sep 2013 13:52:15 -0700 (PDT) In-Reply-To: References: From: Caroline Tice Date: Thu, 05 Sep 2013 20:52:00 -0000 Message-ID: Subject: Re: [PATCH, libvtv] Fix most of the testsuite. To: GCC Patches Content-Type: text/plain; charset=ISO-8859-1 X-IsSubscribed: yes X-SW-Source: 2013-09/txt/msg00336.txt.bz2 Ping? Could somebody please review this for me? -- Caroline Tice cmtice@google.com On Fri, Aug 30, 2013 at 2:55 PM, Caroline Tice wrote: > The attached patch fixes most of the libvtv testsuite tests to run in > the dejagnu test harness. There are still a few tests that I haven't > been able to get working properly under dejagnu yet; they have been > moved to libvtv/testsuite/other-tests for now. As I figure out how to > make them work properly, I will move them out of there and update the > testsuite. This patch looks like it's big, but most of that is files > being moved from one directory to another (so the entire file gets > deleted and then the entire file gets added in again, and nothing in > the file actually changed). > > I have verified that all the tests that are not in the other-tests > subdirectory do run and pass under dejagnu. > > Since I am supposedly the libvtv maintainer, I am not sure how this > patch is supposed to be reviewed...Guidance would be appreciated... > > -- Caroline Tice > cmtice@google.com > > 2013-08-30 Caroline Tice > > * testsuite/event-main.cc: Move to libvtv.cc subdirectory. > * testsuite/environment.cc: Ditto. > * testsuite/template-list2.cc: Ditto. > * testsuite/event.h: Ditto. > * testsuite/dataentry.cc: Ditto. > * testsuite/event-private.h: Ditto. > * testsuite/virtual_inheritance.cc: Ditto. > * testsuite/povray-derived.cc: Ditto. > * testsuite/nested_vcall_test.cc: Ditto. > * testsuite/template-list-iostream.cc: Ditto. > * testsuite/parts-test-extra-parts-views.h: Ditto. > * testsuite/virtfunc-test.cc: Ditto. > * testsuite/parts-test-extra-parts.h: Ditto. > * testsuite/const_vtable.cc: Ditto. > * testsuite/template-list.cc: Ditto. > * testsuite/dup_name.cc: Ditto. > * testsuite/thunk.cc: Ditto. > * testsuite/parts-test-main.h: Ditto. > * testsuite/mul_inh.cc: Ditto. > * testsuite/test1.cc: Ditto. > * testsuite/bb_tests.cc: Ditto. > * testsuite/v8-test-2.cc: Ditto. > * testsuite/thunk_vtable_map_attack.cc: Ditto. > * testsuite/xlan-test.cc: Ditto. > > * testsuite/parts-test-main.cpp: Move to libvtv.cc subdirectory and > change file extension from .cc to .cpp. > * testsuite/event-definitions.cpp: Ditto. > * testsuite/event-main.cpp: Ditto. > * testsuite/derived-main.cpp: Ditto. > * testsuite/derived-lib.cpp: Ditto. > * testsuite/event-private.cpp: Ditto. > * testsuite/parts-test-extra-parts-views.cpp: Ditto. > * testsuite/parts-test-extra-parts.cpp: Ditto. > > * testsuite/parts-test.list: Move to libvtv.cc subdirectory. Change > file extensions inside file from .cc to .cpp. > * testsuite/event.list: Ditto. > * testsuite/derived.list: Ditto. > * testsuite/register_pair.cc: Move to libvtv.cc; rename file to > register_set_pair.cc; include stdlib.h, stdio.h stdint.h string.h > (KEY_TYPE_FIXED_SIZE): New define. > (key_buffer, name_string, fake_names): New global variables. > (generate_names): New function. > (vtv_string_hans): New function. > (main): Add call to generate_names. Update middle for-loop to > initialize new parameters for __VLTRegisterPair... calls; move calls > to __VLTRegisterPair... to middle for-loop. Add calls to > __VLTRegisterSet... > * testsuite/register_pair_mt.cc: Ditto; renamed to > register_set_pair_mt.cc > * testsuite/libvtv.cc/vtv.exp: New file. > * testsuite/libvtv.mempool.cc/mempool.exp: New file. > * testsuite/libvtv.mt.cc/mt.exp: New file. > * testsuite/lib/libvtv.exp: New file. > * testsuite/lib/libvtv-dg.exp: New file. > * testsuite/config/default.exp: New file. > * testsuite/Makefile.am: New file. (Old file was moved to other-tests > subdirectory.) > * testsuite/Makefile.in: New file (generated). > * testsuite/mempool_negative.c: Change to C++ file; move to > libvtv.mempool.cc; include vtv-change-permission.h. > (main): Add call to __VLTChangePermission. > * testsuite/mempool_positive.c: Change to C++ file; move to > libvtv.mempool.cc; include vtv-change-permission.h. > (main): Add call to __VLTChangePermission. > > * testsuite/temp_deriv3.cc: Move to other-tests subdirectory. > * testsuite/environment-fail-64.s: Ditto. > * testsutite/dlopen.cc: Ditto. > * testsuite/so.cc: Ditto. > * testsuite/temp_deriv2.cc: Ditto. > * testsuite/field-test.cc: Ditto. > * testsuite/dlopen_mt.cc: Ditto. > * testsuite/environment-fail-32.s: Ditto. > * testsuite/temp_deriv.cc: Ditto. > * testsuite/replace-fail.cc: Ditto. > * testsuite/other-tests/Makefile.am: New file. Copied from the > Makefile.am that used to be in testsuite directory. > * testsuite/other-tests/Makefile.in: Generated. (New file). > * testsuite/other-tests/README: New file.