[compiler-rt] r260041 - Add coverage tests (defaulted constructors/destructor)

[Xinliang David Li via llvm-commits]

As long as there is a way to do this kind of testing -- I am willing
to migrate all such tests in profile runtime.

thanks,

David


On Mon, Feb 8, 2016 at 11:08 AM, David Blaikie <dblaikie at gmail.com> wrote:
>
>
> On Mon, Feb 8, 2016 at 10:54 AM, Xinliang David Li via llvm-commits
> <llvm-commits at lists.llvm.org> wrote:
>>
>> On Mon, Feb 8, 2016 at 10:46 AM, Vedant Kumar <vsk at apple.com> wrote:
>> > Could this be merged with the test in r260021?
>>
>> In theory 260021 can be merged into this one ( end-to-end test)
>
>
> We don't generally do end-to-end tests in the LIT repo. I realize
> compiler-rt is a bit different, but I think this general attitude still
> holds.
>
> Alexey (for want of any better person to consult - feel free to punt/CC
> others): what's the general philosophy on tests that go into compiler-rt's
> lit suite?
>
>>
>> -- but
>> I think it is better to keep them separate as they are in different
>> repos.  I will clean up the test as you suggested else where.
>>
>> >
>> > Is this a Linux-specific test?
>>
>> It should not be. It is just I don't have a good way to test on other
>> platforms yet.
>>
>> David
>> >
>> > vedant
>> >
>> >> On Feb 7, 2016, at 8:31 AM, Xinliang David Li via llvm-commits
>> >> <llvm-commits at lists.llvm.org> wrote:
>> >>
>> >> Author: davidxl
>> >> Date: Sun Feb  7 10:31:13 2016
>> >> New Revision: 260041
>> >>
>> >> URL: http://llvm.org/viewvc/llvm-project?rev=260041&view=rev
>> >> Log:
>> >> Add coverage tests (defaulted constructors/destructor)
>> >>
>> >> Added:
>> >>    compiler-rt/trunk/test/profile/Linux/coverage_ctors.cpp
>> >>    compiler-rt/trunk/test/profile/Linux/coverage_dtor.cpp
>> >>
>> >> Added: compiler-rt/trunk/test/profile/Linux/coverage_ctors.cpp
>> >> URL:
>> >> http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/profile/Linux/coverage_ctors.cpp?rev=260041&view=auto
>> >>
>> >> ==============================================================================
>> >> --- compiler-rt/trunk/test/profile/Linux/coverage_ctors.cpp (added)
>> >> +++ compiler-rt/trunk/test/profile/Linux/coverage_ctors.cpp Sun Feb  7
>> >> 10:31:13 2016
>> >> @@ -0,0 +1,33 @@
>> >> +// RUN: %clang_profgen -x c++  -std=c++11 -fuse-ld=gold
>> >> -fcoverage-mapping -o %t %s
>> >> +// RUN: env LLVM_PROFILE_FILE=%t.profraw %run %t
>> >> +// RUN: llvm-profdata merge -o %t.profdata %t.profraw
>> >> +// RUN: llvm-cov show %t -instr-profile %t.profdata
>> >> -filename-equivalence 2>&1 | FileCheck %s
>> >> +
>> >> +struct Base {
>> >> +  int B;
>> >> +  Base() : B(2) {}
>> >> +  Base(const struct Base &b2) {
>> >> +    if (b2.B == 0) {
>> >> +      B = b2.B + 1;
>> >> +    } else
>> >> +      B = b2.B;
>> >> +  }
>> >> +};
>> >> +
>> >> +struct Derived : public Base {
>> >> +  Derived(const Derived &) = default; // CHECK:  2| [[@LINE]]|
>> >> Derived
>> >> +  Derived() = default;                // CHECK:  1| [[@LINE]]|
>> >> Derived
>> >> +  int I;
>> >> +  int J;
>> >> +  int getI() { return I; }
>> >> +};
>> >> +
>> >> +Derived dd;
>> >> +int g;
>> >> +int main() {
>> >> +  Derived dd2(dd);
>> >> +  Derived dd3(dd);
>> >> +
>> >> +  g = dd2.getI() + dd3.getI();
>> >> +  return 0;
>> >> +}
>> >>
>> >> Added: compiler-rt/trunk/test/profile/Linux/coverage_dtor.cpp
>> >> URL:
>> >> http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/profile/Linux/coverage_dtor.cpp?rev=260041&view=auto
>> >>
>> >> ==============================================================================
>> >> --- compiler-rt/trunk/test/profile/Linux/coverage_dtor.cpp (added)
>> >> +++ compiler-rt/trunk/test/profile/Linux/coverage_dtor.cpp Sun Feb  7
>> >> 10:31:13 2016
>> >> @@ -0,0 +1,26 @@
>> >> +// RUN: %clang -x c++ -fno-exceptions  -std=c++11 -fuse-ld=gold
>> >> -fprofile-instr-generate -fcoverage-mapping -o %t %s
>> >> +// RUN: env LLVM_PROFILE_FILE=%t.profraw %run %t
>> >> +// RUN: llvm-profdata merge -o %t.profdata %t.profraw
>> >> +// RUN: llvm-cov show %t -instr-profile %t.profdata
>> >> -filename-equivalence 2>&1 | FileCheck %s
>> >> +
>> >> +struct Base {
>> >> +  int B;
>> >> +  Base(int B_) : B(B_) {}
>> >> +  ~Base() {}
>> >> +};
>> >> +
>> >> +struct Derived : public Base {
>> >> +  Derived(int K) : Base(K), I(K), J(K) {}
>> >> +  ~Derived() = default; // CHECK:  2| [[@LINE]]|  ~Derived
>> >> +  int I;
>> >> +  int J;
>> >> +  int getI() { return I; }
>> >> +};
>> >> +
>> >> +int g;
>> >> +int main() {
>> >> +  Derived dd(10);
>> >> +  Derived dd2(120);
>> >> +  g = dd2.getI() + dd.getI();
>> >> +  return 0;
>> >> +}
>> >>
>> >>
>> >> _______________________________________________
>> >> llvm-commits mailing list
>> >> llvm-commits at lists.llvm.org
>> >> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>> >
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at lists.llvm.org
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
>