On Fri, Nov 29, 2013 at 9:02 AM, H.J. Lu <hjl.to...@gmail.com> wrote:
> 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]$
>
I am checking in this as an obvious fix. Tested
on Linux/x86.
--
H.J.
--
diff --git a/gcc/testsuite/lib/asan-dg.exp b/gcc/testsuite/lib/asan-dg.exp
index 823265c..1e6c7dd 100644
--- a/gcc/testsuite/lib/asan-dg.exp
+++ b/gcc/testsuite/lib/asan-dg.exp
@@ -39,9 +39,10 @@ proc asan_link_flags { paths } {
set shlib_ext [get_shlib_extension]
if { $gccpath != "" } {
- append flags " -B${gccpath}/libsanitizer/asan/ "
+ append flags " -B${gccpath}/libsanitizer/ "
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"
