[PATCH] [test-suite] Add test for basic correctness of frame lowering

Renato Golin renato.golin at linaro.org
Thu Apr 23 09:32:16 PDT 2015


Hi Kristof,

I haven't looked in detail what the whole code does, but I can see how it stress the stack and how the test works, roughly. As this is in a new directory, it'll be easy to disable on targets where it doesn't work yet, or on compilers that don't support.

Apart from my comment, I think this is a good addition to the test-suite.

cheers,
--renato


================
Comment at: MultiSource/Makefile:5
@@ -4,3 +4,1 @@
 PARALLEL_DIRS  := Applications Benchmarks
-# For now only Mips uses UnitTests
-ifeq ($(ARCH),Mips)
----------------
This is a bit too generic. I'd add AArch64, maybe ARM, but not enable for everything before making sure all UnitTests work on all platforms we run the test-suite.

http://reviews.llvm.org/D8922

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list