hpac / ttc-c Goto Github PK

In #4 I got a problem, that my version of TTC was not the correct version, and therefore I got a compilation error. But how should I have known, because TTC-C neither does bring the correct version of TTC, not does it tell, which version I need.

My suggestion is to add TTC as a subtree into this repository. So that when I update this repository, I automatically have the correct version of TTC installed. When ttc is also executed from this local directory, and does define TTC_ROOT for it, it simplifies the installation process much further.

I know this is an issue tracker, and not really the right place to ask question, but I think I do it anyway.

With ttc_init I get a pointer to a ttc handler struct. Am I allowed to have several of them around, and is this behavior encouraged? Is it ok to have one struct in each module? Is it ok to have one instance per parameter? Or do I have to keep one single instance around and put it into some global variable?

with ttc_set_opt I can set the dataType as a parameter of the handle. Assuming that there is only one handle around, this is like a global state. I don't think this is a very wise idea, because as soon as I have code that uses functions with different dataTypes, I have to set the datatype before any call into ttc. If I forget it anywhere, this code could be broken anytime because it depends on some global state which is not constant. I could even lead to some defensive code that queries the dataType parameter, changes it calls into tcc and then reverts it back, just to ensure that no code could potentially be broken.

This could completely be avoided, by simply putting datatype into the param.

I have a compile error in a simple example that I've written, and I can't figure out what I did wrong, since the compilation problem doesn't really help me what went wrong.

this is the error I got during execution:

sTranspose_10_2x12_par.cpp(19): error: no instance of function template "sTranspose_10_2x12_par" matches the argument list
            argument types are: (const float *, float *, float, float, const int *, const int *)
      sTranspose_10_2x12_par<2, 3, 4>((const TENSOR_IN_T *)input,
      ^

icpc: error #10236: File not found:  'sTranspose_10_2x12_par.o'
icpc: command line error: no files specified; for help type "icpc -help"

The test program I've written to produce this bug is here:
https://github.com/krux02/TTC-C-test/blob/master/main.c

when LD_LIBRARY_PATH, LIBRARY_PATH and INCLUDE_PATH are set correctly, make && ./a.out is enough to reproduce the bug.