[PATCH] D15999: Adding doxygen comments to the LLVM intrinsics (part 2, _wmmintrin_pclmul.h)

Sean Silva via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 19 20:04:05 PST 2016


silvas added a subscriber: silvas.
silvas added a comment.

This may sound stupid, but: can you benchmark the time it takes to build some project (that actually uses intrinsics in most translation units, e.g. a game) with the headers w/ and w/o the doxygen comments to check that all the extra comment skipping doesn't affect compilation time? I.e. run your script to add the comments for "all" the intrinsic headers (similar to what you expect the final state to be after all these patches) and test the build time of a game (and compare with the unmodified headers).

Also, can you post a patch that changes "all" the headers to have doxygen comments like you intend, so that others can test and verify?


Repository:
  rL LLVM

http://reviews.llvm.org/D15999





More information about the cfe-commits mailing list