[PATCH] D15792: [AArch64] Turn off PredictableSelectIsExpensive on the Cortex-A57

Matthias Braun via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 7 12:55:54 PST 2016


As we are on the topic of SPEC and test-suite: The SPEC running Makefiles in the test-suite do not come close to proper SPEC runs. They only ever run 1 input while SPEC often supplies multiple different inputs. They are good as “smoke” tests but do not necessarily reflect the changes in a real ref run.
If you need spec-ref numbers then I would recommend to use the runspec tool that comes with SPEC or the test-suite/LNTBased stuff (lnt runtest nt —spec-with-ref I believe). I’m also currently integrating proper SPEC support into the new cmake/lit style of the test-suite which already properly handle SPEC95 and SPEC2000 but the SPEC2006 part isn’t finished yet.

- Matthias

> On Jan 7, 2016, at 3:56 AM, Junmo Park via llvm-commits <llvm-commits at lists.llvm.org> wrote:
> 
> flyingforyou added a comment.
> 
> Thanks for confirmation James.
> 
> Evandro, you're right. As you said, when we run spec2000,2006 default `RUN_TYPE` in test-suite is not ref.
> When I looked https://www.spec.org/cpu2000/docs/runspec.html#toc_I.C. , I found ref is default setting.
> 
> I will update data again.
> 
> James, I still think this commit might be worth for Cortex-A57. Could you give me a second chance, please?
> 
> Junmo.
> 
> 
> http://reviews.llvm.org/D15792
> 
> 
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits



More information about the llvm-commits mailing list