From 31c7abcc27a33d63ac8d335387a0ce7b3ad74954 Mon Sep 17 00:00:00 2001 From: jsmall-nvidia Date: Wed, 18 Sep 2019 11:40:59 -0400 Subject: Improvements to testing and ABI for CPU (#1057) * WIP: Improving CPU performance/ABI * Optionally output code on CPU for groupThreadID and groupID. * Added ability to set compute dispatch size on command line for render-test. Dispatch compute tests taking into account dispatch size. Added test for semantics are working. * Test using GroupRange. * Fix problem with adding \n for externa diagnostic - to do it if there isn't a \n at the end. Change the ouput order (put result before) so last value is diagnostic string. * Made GroupRange the default exposed CPU ABI entry point style. Removed CPU_EXECUTE test style -as tested via the now cross platform render-test * Split out execution from setup for execution to improve perf. * For better code coverage/testing test all styles of CPU compute entry point. * Improve documentation for ABI changes for CPU code. Add 'expecting' to error message from review. * Fix small typos. --- tools/render-test/options.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'tools/render-test/options.cpp') diff --git a/tools/render-test/options.cpp b/tools/render-test/options.cpp index d2f21a5d9..e13a2b88f 100644 --- a/tools/render-test/options.cpp +++ b/tools/render-test/options.cpp @@ -183,7 +183,7 @@ SlangResult parseOptions(int argc, const char*const* argv, Slang::WriterHelper s { if (argCursor == argEnd) { - stdError.print("error: comma separated compute dispatch size for '%s'\n", arg); + stdError.print("error: expecting a comma separated compute dispatch size for '%s'\n", arg); return SLANG_FAIL; } List slices; -- cgit v1.2.3