kromanova added a comment. In http://reviews.llvm.org/D15999#330794, @silvas wrote:
> 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? Out of curiosity, do you know if the impact to the build time for Eric's change of starting to use the target attributes instead of conditional inclusion was measured (r239883) on a large scale application. If so, what were the results? I should probably include Eric. Repository: rL LLVM http://reviews.llvm.org/D15999 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
