[PATCH] D55484: ComputeLineNumbers: delete SSE2 vectorization

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 10 10:08:22 PST 2018


MaskRay added a comment.

In D55484#1324983 <https://reviews.llvm.org/D55484#1324983>, @bkramer wrote:

> The performance difference on preprocessing huge files was tiny back then, doesn't surprise me that it disappeared. What did you test this on?


I tested it on

  cat lib/Sema/*.cpp lib/CodeGen/*.cpp > /tmp/all.cpp
  perf stat -r 10 clang -E /tmp/all.cpp [-I extracted from build.ninja]

and `/tmp/all.cpp` (13M) repeated 3 and 9 times.


Repository:
  rC Clang

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

https://reviews.llvm.org/D55484





More information about the cfe-commits mailing list