[PATCH] D21593: test-suite: Remove lemon benchmark

Kristof Beyls via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 21 23:26:07 PDT 2016


kristof.beyls added a comment.

Thanks for the analysis!

Interestingly, on the systems I look at most often (linux AArch64 and AArch32), I don't see lemon produce too much noise.
Anyway, with your analysis, it's clear that lemon doesn't really test quality of compiler code generation, so I agree not to run it as a benchmark.

However, it may still make sense to keep this in the test-suite as a correctness test, in which case it should be just listed under "if(NOT TEST_SUITE_BENCHMARKING_ONLY)"?

Thanks,

Kristof


Repository:
  rL LLVM

http://reviews.llvm.org/D21593





More information about the llvm-commits mailing list