r178061 - clang/test/CodeGenCXX/debug-info-namespace.cpp: Disable it on non-bash lit for now.

NAKAMURA Takumi geek4civic at gmail.com
Tue Mar 26 16:48:12 PDT 2013


AFAIK, it's not in bugzila. Yes, this behavior can be a bug.

For now, I'll try to canonicalize paths in somewhere. After then, I'll
propose to unify internal separator (in win32) as not '\\', but '/'.
It has been always nasty to me.

...Takumi

2013/3/27 Jordan Rose <jordan_rose at apple.com>:
> !8 is unexpected? !1 has the wrong path separator. I get the feeling there's an actual bug here...is there a Bugzilla report yet?
>
> Jordan
>
>
> On Mar 26, 2013, at 11:57 , NAKAMURA Takumi <geek4civic at gmail.com> wrote:
>
>> Author: chapuni
>> Date: Tue Mar 26 13:57:40 2013
>> New Revision: 178061
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=178061&view=rev
>> Log:
>> clang/test/CodeGenCXX/debug-info-namespace.cpp: Disable it on non-bash lit for now.
>>
>> With dosish filename, it misgenerates an improper extra entry.
>>
>> !1 = metadata !{metadata !"E:\5Cllvm\5Cllvm-project\5Cclang\5Ctest\5CCodeGenCXX/debug-info-namespace.cpp", metadata !"E:\5Cllvm\5Cbuild\5Cninja-win32-vs11\5Ctools\5Cclang\5Ctest\5CCodeGenCXX"}
>>
>> !8 = metadata !{metadata !"E:\5Cllvm\5Cllvm-project\5Cclang\5Ctest\5CCodeGenCXX\5Cdebug-info-namespace.cpp", metadata !"E:\5Cllvm\5Cbuild\5Cninja-win32-vs11\5Ctools\5Cclang\5Ctest\5CCodeGenCXX"}
>>
>> !8 is unexpected.
>>
>> Modified:
>>    cfe/trunk/test/CodeGenCXX/debug-info-namespace.cpp
>>
>> Modified: cfe/trunk/test/CodeGenCXX/debug-info-namespace.cpp
>> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGenCXX/debug-info-namespace.cpp?rev=178061&r1=178060&r2=178061&view=diff
>> ==============================================================================
>> --- cfe/trunk/test/CodeGenCXX/debug-info-namespace.cpp (original)
>> +++ cfe/trunk/test/CodeGenCXX/debug-info-namespace.cpp Tue Mar 26 13:57:40 2013
>> @@ -12,3 +12,6 @@ int i;
>> // CHECK: [[NS]] = {{.*}}, metadata [[FILE2:![0-9]*]], metadata [[CTXT:![0-9]*]], {{.*}} ; [ DW_TAG_namespace ] [B] [line 1]
>> // CHECK: [[CTXT]] = {{.*}}, metadata [[FILE]], null, {{.*}} ; [ DW_TAG_namespace ] [A] [line 3]
>> // CHECK: [[FILE2]]} ; [ DW_TAG_file_type ] [{{.*}}foo.cpp]
>> +
>> +// FIXME: It is confused on win32 to generate file entry when dosish filename is given.
>> +// REQUIRES: shell
>>
>>
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>



More information about the cfe-commits mailing list