public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [patch, testsuite] Fix ubsan for testing when libstdc++ isn't installed
@ 2014-12-20 18:09 FX
  2014-12-22 17:14 ` Mike Stump
  0 siblings, 1 reply; 4+ messages in thread
From: FX @ 2014-12-20 18:09 UTC (permalink / raw)
  To: GCC Patches

This patch below allows ubsan to run when libstdc++ is built but not installed (something which happens on darwin, in particular). This fixes all 658 ubsan failures when run in this particular configuration.

OK to commit?
FX



2014-12-20  Francois-Xavier Coudert  <fxcoudert@gcc.gnu.org>

	* lib/ubsan-dg.exp: Add library path for libstdc++.


Index: gcc/testsuite/lib/ubsan-dg.exp
===================================================================
--- gcc/testsuite/lib/ubsan-dg.exp	(revision 218989)
+++ gcc/testsuite/lib/ubsan-dg.exp	(working copy)
@@ -47,6 +47,7 @@ proc ubsan_link_flags { paths } {
 	  append flags " -B${gccpath}/libsanitizer/ubsan/ "
 	  append flags " -L${gccpath}/libsanitizer/ubsan/.libs"
 	  append ld_library_path ":${gccpath}/libsanitizer/ubsan/.libs"
+	  append ld_library_path ":${gccpath}/libstdc++-v3/src/.libs"
       }
     } else {
       global tool_root_dir

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [patch, testsuite] Fix ubsan for testing when libstdc++ isn't installed
  2014-12-20 18:09 [patch, testsuite] Fix ubsan for testing when libstdc++ isn't installed FX
@ 2014-12-22 17:14 ` Mike Stump
  2015-02-13 22:24   ` Jack Howarth
  0 siblings, 1 reply; 4+ messages in thread
From: Mike Stump @ 2014-12-22 17:14 UTC (permalink / raw)
  To: FX; +Cc: GCC Patches

On Dec 20, 2014, at 8:58 AM, FX <fxcoudert@gmail.com> wrote:
> This patch below allows ubsan to run when libstdc++ is built but not installed (something which happens on darwin, in particular). This fixes all 658 ubsan failures when run in this particular configuration.
> 
> OK to commit?

Ok.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [patch, testsuite] Fix ubsan for testing when libstdc++ isn't installed
  2014-12-22 17:14 ` Mike Stump
@ 2015-02-13 22:24   ` Jack Howarth
  2015-02-15 19:03     ` Mike Stump
  0 siblings, 1 reply; 4+ messages in thread
From: Jack Howarth @ 2015-02-13 22:24 UTC (permalink / raw)
  To: Mike Stump; +Cc: FX, GCC Patches

Mike and FX,
     Shouldn't we also apply...

Author: fxcoudert
Date: Mon Dec 22 21:57:45 2014
New Revision: 219035

URL: https://gcc.gnu.org/viewcvs?rev=219035&root=gcc&view=rev
Log:
* lib/ubsan-dg.exp: Add library path for libstdc++.

Modified:
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/lib/ubsan-dg.exp

to gcc-4_9-branch for 4.9.3?

https://gcc.gnu.org/ml/gcc-testresults/2015-02/msg01535.html

shows that we need it.
                Jack

On Mon, Dec 22, 2014 at 12:02 PM, Mike Stump <mikestump@comcast.net> wrote:
> On Dec 20, 2014, at 8:58 AM, FX <fxcoudert@gmail.com> wrote:
>> This patch below allows ubsan to run when libstdc++ is built but not installed (something which happens on darwin, in particular). This fixes all 658 ubsan failures when run in this particular configuration.
>>
>> OK to commit?
>
> Ok.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [patch, testsuite] Fix ubsan for testing when libstdc++ isn't installed
  2015-02-13 22:24   ` Jack Howarth
@ 2015-02-15 19:03     ` Mike Stump
  0 siblings, 0 replies; 4+ messages in thread
From: Mike Stump @ 2015-02-15 19:03 UTC (permalink / raw)
  To: Jack Howarth; +Cc: FX, GCC Patches

On Feb 13, 2015, at 2:24 PM, Jack Howarth <howarth.at.gcc@gmail.com> wrote:
> Mike and FX,
>     Shouldn't we also apply…

Ok.

> Author: fxcoudert
> Date: Mon Dec 22 21:57:45 2014
> New Revision: 219035
> 
> URL: https://gcc.gnu.org/viewcvs?rev=219035&root=gcc&view=rev
> Log:
> * lib/ubsan-dg.exp: Add library path for libstdc++.
> 
> Modified:
>    trunk/gcc/testsuite/ChangeLog
>    trunk/gcc/testsuite/lib/ubsan-dg.exp
> 
> to gcc-4_9-branch for 4.9.3?
> 
> https://gcc.gnu.org/ml/gcc-testresults/2015-02/msg01535.html
> 
> shows that we need it.
>                Jack
> 
> On Mon, Dec 22, 2014 at 12:02 PM, Mike Stump <mikestump@comcast.net> wrote:
>> On Dec 20, 2014, at 8:58 AM, FX <fxcoudert@gmail.com> wrote:
>>> This patch below allows ubsan to run when libstdc++ is built but not installed (something which happens on darwin, in particular). This fixes all 658 ubsan failures when run in this particular configuration.
>>> 
>>> OK to commit?
>> 
>> Ok.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2015-02-15 19:03 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-12-20 18:09 [patch, testsuite] Fix ubsan for testing when libstdc++ isn't installed FX
2014-12-22 17:14 ` Mike Stump
2015-02-13 22:24   ` Jack Howarth
2015-02-15 19:03     ` Mike Stump

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).