aleksandr.urakov added a comment.
In https://reviews.llvm.org/D47708#1125029, @labath wrote:
> Thank you for implementing the lldb-test extension. Now that we have that,
> and the `/order`, we should be able to get rid of the binaries for the
> function-level-linking test. You should be able to rewrite it into something
> like this:
>
> lit/SymbolFile/PDB/function-level-linking.cpp:
>
> // REQUIRES: windows lld
> //
> // RUN: clang-cl /c /Zi /Gy %s /o %t
> // RUN: lld-link /debug:full /nodefaultlib /entry:main
> order:@%S/Inputs/function-level-linking.ord %t.obj /out:%t
> // RUN: lldb-test symbols -verify %t
>
> #include "function-level-linking.h" // You'll probably need to adjust the
> include path for this to work.
>
> int foo() {
> return 0;
> }
>
> int main() {
> return foo() + bar() + baz();
> }
>
>
> As for the second test, does it actually bring anything new to the table? As
> far as I can tell, the line table parsing code does not actually use any
> information about functions. Combined with the fact that we don't know how to
> write the test without checked in binaries, maybe we could just drop it ?
Yes, you are right, the second test does not bring anything new to the table. I
have made it just by Leonard Mosescu suggestion. I think we can drop it.
As for `/ORDER` switch, I have tried it on the release version. Thanks for the
point, I'll try the master version.
https://reviews.llvm.org/D47708
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
