diff mbox

testsuite: Register loaded libs

Message ID 1398620714-1331-1-git-send-email-sebastian.huber@embedded-brains.de
State New
Headers show

Commit Message

Sebastian Huber April 27, 2014, 5:45 p.m. UTC 
libffi/ChangeLog
2014-04-27  Sebastian Huber  <sebastian.huber@embedded-brains.de>

	* testsuite/lib/libffi.exp (load_gcc_lib): Register loaded libs.

libjava/ChangeLog
2014-04-27  Sebastian Huber  <sebastian.huber@embedded-brains.de>

	* testsuite/lib/libjava.exp (load_gcc_lib): Register loaded
	libs.

libstdc++-v3/ChangeLog
2014-04-27  Sebastian Huber  <sebastian.huber@embedded-brains.de>

	* testsuite/lib/libstdc++.exp (load_gcc_lib): Register loaded
	libs.
---
 libffi/testsuite/lib/libffi.exp          | 4 +++-
 libjava/testsuite/lib/libjava.exp        | 4 +++-
 libstdc++-v3/testsuite/lib/libstdc++.exp | 4 +++-
 3 files changed, 9 insertions(+), 3 deletions(-)

Comments

Mike Stump April 28, 2014, 2:23 a.m. UTC | #1 
On Apr 27, 2014, at 10:45 AM, Sebastian Huber <sebastian.huber@embedded-brains.de> wrote:
> 2014-04-27  Sebastian Huber  <sebastian.huber@embedded-brains.de>
> 
> 	* testsuite/lib/libffi.exp (load_gcc_lib): Register loaded libs.

So, I didn’t see anything that strikes me as wrong, but, I’m curious why you want this?  I didn’t see any uses?
Sebastian Huber April 28, 2014, 7:32 a.m. UTC | #2 
On 2014-04-28 04:23, Mike Stump wrote:
> On Apr 27, 2014, at 10:45 AM, Sebastian Huber <sebastian.huber@embedded-brains.de> wrote:
>> 2014-04-27  Sebastian Huber  <sebastian.huber@embedded-brains.de>
>>
>> 	* testsuite/lib/libffi.exp (load_gcc_lib): Register loaded libs.
>
> So, I didn’t see anything that strikes me as wrong, but, I’m curious why you want this?  I didn’t see any uses?
>

I would like to add tests for C/C++ compatibility of atomic operations.

http://gcc.gnu.org/ml/gcc/2014-04/msg00267.html

This patch unifies all copy and paste versions of load_gcc_lib.
Mike Stump April 28, 2014, 4:24 p.m. UTC | #3 
On Apr 28, 2014, at 12:32 AM, Sebastian Huber <sebastian.huber@embedded-brains.de> wrote:
> On 2014-04-28 04:23, Mike Stump wrote:
>> On Apr 27, 2014, at 10:45 AM, Sebastian Huber <sebastian.huber@embedded-brains.de> wrote:
>>> 2014-04-27  Sebastian Huber  <sebastian.huber@embedded-brains.de>
>>> 
>>> 	* testsuite/lib/libffi.exp (load_gcc_lib): Register loaded libs.
>> 
>> So, I didn’t see anything that strikes me as wrong, but, I’m curious why you want this?  I didn’t see any uses?
>> 
> 
> I would like to add tests for C/C++ compatibility of atomic operations.
> 
> http://gcc.gnu.org/ml/gcc/2014-04/msg00267.html
> 
> This patch unifies all copy and paste versions of load_gcc_lib.

Ah, thanks for the pointer…

Ok.
Sebastian Huber May 9, 2014, 8:36 a.m. UTC | #4 
On 2014-04-28 18:24, Mike Stump wrote:
> On Apr 28, 2014, at 12:32 AM, Sebastian Huber<sebastian.huber@embedded-brains.de>  wrote:
>> >On 2014-04-28 04:23, Mike Stump wrote:
>>> >>On Apr 27, 2014, at 10:45 AM, Sebastian Huber<sebastian.huber@embedded-brains.de>  wrote:
>>>> >>>2014-04-27  Sebastian Huber<sebastian.huber@embedded-brains.de>
>>>> >>>
>>>> >>>	* testsuite/lib/libffi.exp (load_gcc_lib): Register loaded libs.
>>> >>
>>> >>So, I didn’t see anything that strikes me as wrong, but, I’m curious why you want this?  I didn’t see any uses?
>>> >>
>> >
>> >I would like to add tests for C/C++ compatibility of atomic operations.
>> >
>> >http://gcc.gnu.org/ml/gcc/2014-04/msg00267.html
>> >
>> >This patch unifies all copy and paste versions of load_gcc_lib.
> Ah, thanks for the pointer…
>
> Ok.

Can someone please commit this for me.
Chung-Ju Wu May 20, 2014, 8:32 a.m. UTC | #5 
2014-05-09 16:36 GMT+08:00 Sebastian Huber <sebastian.huber@embedded-brains.de>:
> On 2014-04-28 18:24, Mike Stump wrote:
>> On Apr 28, 2014, at 12:32 AM, Sebastian
>> Huber<sebastian.huber@embedded-brains.de>  wrote:
>>>
>>> >I would like to add tests for C/C++ compatibility of atomic operations.
>>> >
>>> >http://gcc.gnu.org/ml/gcc/2014-04/msg00267.html
>>> >
>>> >This patch unifies all copy and paste versions of load_gcc_lib.
>>
>> Ah, thanks for the pointer…
>>
>> Ok.
>
>
> Can someone please commit this for me.
>

Hi, Sebastian,

I committed this patch for you:
  https://gcc.gnu.org/r210634


Best regards,
jasonwucj
diff mbox

Patch

diff --git a/libffi/testsuite/lib/libffi.exp b/libffi/testsuite/lib/libffi.exp
index 3c61baa..25efce2 100644
--- a/libffi/testsuite/lib/libffi.exp
+++ b/libffi/testsuite/lib/libffi.exp
@@ -15,8 +15,10 @@ 
 # <http://www.gnu.org/licenses/>.
 
 proc load_gcc_lib { filename } {
-    global srcdir
+    global srcdir loaded_libs
+
     load_file $srcdir/../../gcc/testsuite/lib/$filename
+    set loaded_libs($filename) ""
 }
 
 load_lib dg.exp
diff --git a/libjava/testsuite/lib/libjava.exp b/libjava/testsuite/lib/libjava.exp
index 0de823b..0cfb253 100644
--- a/libjava/testsuite/lib/libjava.exp
+++ b/libjava/testsuite/lib/libjava.exp
@@ -2,8 +2,10 @@ 
 # Free Software Foundation
 
 proc load_gcc_lib { filename } {
-    global srcdir
+    global srcdir loaded_libs
+
     load_file $srcdir/../../gcc/testsuite/lib/$filename
+    set loaded_libs($filename) ""
 }
 
 load_lib libgloss.exp
diff --git a/libstdc++-v3/testsuite/lib/libstdc++.exp b/libstdc++-v3/testsuite/lib/libstdc++.exp
index 95954d8..a23ea3b 100644
--- a/libstdc++-v3/testsuite/lib/libstdc++.exp
+++ b/libstdc++-v3/testsuite/lib/libstdc++.exp
@@ -38,8 +38,10 @@ 
 # ~/.dejagnurc or $DEJAGNU.
 
 proc load_gcc_lib { filename } {
-    global srcdir
+    global srcdir loaded_libs
+
     load_file $srcdir/../../gcc/testsuite/lib/$filename
+    set loaded_libs($filename) ""
 }
 
 # system routines