Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2014-04-21T23:27:22Z

Verilator only supports strings in a few limited cases. Might you attempt a patch? This one houldn't be too bad in V3Width 's AstSFormat visitor look for 's' with the corresponding node having a dtypep->basicp of AstBasicDTypeKwd::STRING, change the code to %S, then in V3Emit use %S to mean add a ".c_ptr()" when printing the nodep.

Original Redmine Comment
Author Name: Glen Gibb
Original Date: 2014-04-22T02:16:16Z

I spent a little time playing with this and it looks like the code will require quite a bit more work.

The problem is that the generated C code assigns integers to string objects. For example, the attached .v file includes the following assignment in the resulting .cpp file:

     // in Vt_string_print.h:
     string v__DOT__msg;

     // in Vt_string_print.cpp:
     vlTOPp->v__DOT__msg = ((2 == vlTOPp->v__DOT__cyc)
			    ? VL_ULL(0x74776f) : VL_ULL(0x6e6f742074776f));

As you can see, the code is assigning my two strings as integers to the string object. This ends up assigning only the least-significant byte (0x6f) to my string object, not the sequence of bytes.

Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2014-04-22T11:08:10Z

I guess V3Width will need to be taught how to do string promotion. It's best if you wait until my present set of edits is in. Then you'd see that there's a dtype of string being assigned a dtype of a number, and would convert to a AstConstString.

Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2014-11-28T20:02:24Z

Fixed in git towards 3.867.

Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2014-12-20T21:52:36Z

In 3.868.