On Fri, Nov 29, 2013 at 8:47 AM, H.J. Lu <hjl.to...@gmail.com> wrote:
> On Fri, Nov 29, 2013 at 4:31 AM, Yury Gribov <y.gri...@samsung.com> wrote:
>>> Please fix up formatting
>>
>>> Also, isn't libsanitizer/libsanitizer.spec.in missing from the patch?
>>>
>>> Ok with that fixed.
>>
>>  Thanks, r205524
>>
>
> On Linux/x86, I got
>
> FAIL: c-c++-common/asan/pr59063-2.c  -O0  (test for excess errors)
> FAIL: c-c++-common/asan/pr59063-2.c  -O0  (test for excess errors)
> FAIL: c-c++-common/asan/pr59063-2.c  -O1  (test for excess errors)
> FAIL: c-c++-common/asan/pr59063-2.c  -O1  (test for excess errors)
> FAIL: c-c++-common/asan/pr59063-2.c  -O2 -flto -fno-use-linker-plugin
> -flto-partition=none  (test for excess errors)
> FAIL: c-c++-common/asan/pr59063-2.c  -O2 -flto -fno-use-linker-plugin
> -flto-partition=none  (test for excess errors)
> FAIL: c-c++-common/asan/pr59063-2.c  -O2 -flto -fuse-linker-plugin
> -fno-fat-lto-objects  (test for excess errors)
> FAIL: c-c++-common/asan/pr59063-2.c  -O2 -flto -fuse-linker-plugin
> -fno-fat-lto-objects  (test for excess errors)
> FAIL: c-c++-common/asan/pr59063-2.c  -O2  (test for excess errors)
> FAIL: c-c++-common/asan/pr59063-2.c  -O2  (test for excess errors)
> FAIL: c-c++-common/asan/pr59063-2.c  -O3 -fomit-frame-pointer  (test
> for excess errors)
> FAIL: c-c++-common/asan/pr59063-2.c  -O3 -fomit-frame-pointer  (test
> for excess errors)
> FAIL: c-c++-common/asan/pr59063-2.c  -O3 -g  (test for excess errors)
> FAIL: c-c++-common/asan/pr59063-2.c  -O3 -g  (test for excess errors)
> FAIL: c-c++-common/asan/pr59063-2.c  -Os  (test for excess errors)
> FAIL: c-c++-common/asan/pr59063-2.c  -Os  (test for excess errors)
>

This change

diff --git a/gcc/testsuite/lib/asan-dg.exp b/gcc/testsuite/lib/asan-dg.exp
index e0bf2da..823265c 100644
--- a/gcc/testsuite/lib/asan-dg.exp
+++ b/gcc/testsuite/lib/asan-dg.exp
@@ -39,9 +39,9 @@ proc asan_link_flags { paths } {
     set shlib_ext [get_shlib_extension]

     if { $gccpath != "" } {
+      append flags " -B${gccpath}/libsanitizer/asan/ "
       if { [file exists "${gccpath}/libsanitizer/asan/.libs/libasan.a"]
      || [file exists
"${gccpath}/libsanitizer/asan/.libs/libasan.${shlib_ext}"] } {
-    append flags " -B${gccpath}/libsanitizer/asan/ "
     append flags " -L${gccpath}/libsanitizer/asan/.libs "
     append ld_library_path ":${gccpath}/libsanitizer/asan/.libs"
       }

is wrong since

[hjl@gnu-6 x86_64-unknown-linux-gnu]$ find libsanitizer -name libsanitizer.spec
libsanitizer/libsanitizer.spec
[hjl@gnu-6 x86_64-unknown-linux-gnu]$

-- 
H.J.

Reply via email to