<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 8, 2016 at 11:14 AM, Xinliang David Li via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: davidxl<br>
Date: Mon Feb  8 13:14:14 2016<br>
New Revision: 260126<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=260126&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=260126&view=rev</a><br>
Log:<br>
Simplify test cases<br></blockquote><div><br>It's handy to mention that code review changes relate to the original commit in the commit message (mentioning at least the revision of the original commit, and possibly the reviewer who suggested the change can be handy).<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Modified:<br>
    cfe/trunk/test/Profile/def-ctors.cpp<br>
    cfe/trunk/test/Profile/def-dtors.cpp<br>
<br>
Modified: cfe/trunk/test/Profile/def-ctors.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Profile/def-ctors.cpp?rev=260126&r1=260125&r2=260126&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Profile/def-ctors.cpp?rev=260126&r1=260125&r2=260126&view=diff</a><br>
==============================================================================<br>
--- cfe/trunk/test/Profile/def-ctors.cpp (original)<br>
+++ cfe/trunk/test/Profile/def-ctors.cpp Mon Feb  8 13:14:14 2016<br>
@@ -5,12 +5,7 @@<br>
 struct Base {<br>
   int B;<br>
   Base() : B(2) {}<br>
-  Base(const struct Base &b2) {<br>
-    if (b2.B == 0) {<br>
-      B = b2.B + 1;<br>
-    } else<br>
-      B = b2.B;<br>
-  }<br>
+  Base(const struct Base &b2) {}<br>
 };<br>
<br>
 struct Derived : public Base {<br>
@@ -28,18 +23,14 @@ struct Derived : public Base {<br>
   // Check that coverage mapping has 6 function records including<br>
   // the defaulted Derived::Derived(const Derived), and Derived::Derived()<br>
   // methds.<br>
-  // COVMAP: @__llvm_coverage_mapping = {{.*}} { { i32, i32, i32, i32 }, [6 x<br>
+  // COVMAP: @__llvm_coverage_mapping = {{.*}} { { i32, i32, i32, i32 }, [5 x</blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
   // <{{.*}}>],<br>
-  int I;<br>
-  int J;<br>
-  int getI() { return I; }<br>
 };<br>
<br>
 Derived dd;<br>
 int g;<br>
 int main() {<br>
   Derived dd2(dd);<br>
-<br>
-  g = dd2.getI();<br>
+  g = dd2.B;<br>
   return 0;<br>
 }<br>
<br>
Modified: cfe/trunk/test/Profile/def-dtors.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Profile/def-dtors.cpp?rev=260126&r1=260125&r2=260126&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Profile/def-dtors.cpp?rev=260126&r1=260125&r2=260126&view=diff</a><br>
==============================================================================<br>
--- cfe/trunk/test/Profile/def-dtors.cpp (original)<br>
+++ cfe/trunk/test/Profile/def-dtors.cpp Mon Feb  8 13:14:14 2016<br>
@@ -9,7 +9,7 @@ struct Base {<br>
 };<br>
<br>
 struct Derived : public Base {<br>
-  Derived(int K) : Base(K), I(K), J(K) {}<br>
+  Derived(int K) : Base(K) {}<br>
   ~Derived() = default;<br>
   // PGOGEN-LABEL: define {{.*}}@_ZN7DerivedD2Ev<br>
   // PGOGEN: %pgocount = load {{.*}} @__profc__ZN7DerivedD2Ev<br>
@@ -18,18 +18,13 @@ struct Derived : public Base {<br>
<br>
   // Check that coverage mapping has 6 function records including<br>
   // the default destructor in the derived class.<br>
-  // COVMAP: @__llvm_coverage_mapping = {{.*}} { { i32, i32, i32, i32 }, [6 x<br>
+  // COVMAP: @__llvm_coverage_mapping = {{.*}} { { i32, i32, i32, i32 }, [5 x<br>
   // <{{.*}}>],<br>
-<br>
-  int I;<br>
-  int J;<br>
-  int getI() { return I; }<br>
 };<br>
<br>
-Derived dd(100);<br>
-int g;<br>
 int main() {<br>
-  Derived dd2(dd.getI());<br>
-  g = dd2.getI();<br>
+  Derived dd2(10);<br>
+  if (dd2.B != 10)<br>
+    return 1;<br>
   return 0;<br>
 }<br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
</blockquote></div><br></div></div>