[PATCH] D20993: Add support for collating profiles for use with code coverage

[Vedant Kumar via llvm-commits]

vsk added inline comments.

================
Comment at: utils/prepare-code-coverage-artifact.py:20
@@ +19,3 @@
+    raw_profiles = glob.glob(os.path.join(profile_data_dir, '*.profraw'))
+    manifest_path = os.path.join(profile_data_dir, 'profiles.manifest')
+    profdata_path = os.path.join(profile_data_dir, 'Coverage.profdata')
----------------
davidxl wrote:
> is profiles.manifest file needed here? does it simplify the process?
It's a bit of future-proofing. It was needed with the previous patch because of the sheer number of raw profiles. Now, it helps avoid failures during the `check_call` step due to too many arguments.


http://reviews.llvm.org/D20993