ellis accepted this revision.
ellis added a comment.
This revision is now accepted and ready to land.

Thanks for adding the test case! Can you also link the original diff in the 
summary?



================
Comment at: compiler-rt/test/profile/instrprof-merge-empty-data.test:1
+// Test 
+// RUN: rm -rf %t.dir && split-file %s %t.dir && cd %t.dir
----------------
Please add a more descriptive explanation for this test 😜


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D159000/new/

https://reviews.llvm.org/D159000

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to