[PATCH] D53373: [CodeMetrics] Don't let extends of i1 be free.

[Jonas Paulsson via Phabricator via llvm-commits]

jonpa added a reviewer: fhahn.
jonpa added a subscriber: fhahn.
jonpa added a comment.

I ran the test suite in benchmarking mode (using http://llvm.org/docs/lnt/quickstart.html) on my Intel laptop (i7 4 cores/8 threads), following all the advice including turning frequency scaling and turbo boost mode off. With 5 + 5 runs, I could not see any runtime improvements / regressions at all (When I did 2 +2 runs while using the laptop for editing I saw just a few minor cases both ways).

This should indicate that this patch would be ok for x86 at least. Of course it wouldn't hurt if someone tried some other benchmarks as well for X86, and hopefully someone could do this also for some other target. @fhahn: Could you perhaps try this on Arm?


https://reviews.llvm.org/D53373