DST coercions and DST fields in structs

The commits are not quite stand alone, I should probably squash them together before landing. In particular if you review the individual commits, then you'll see some scrappy stuff that gets fixed in later commits. But reading the commits in order might be easier to get an overall idea of what is going on.

The first commit includes putting back time zone into our time library - @pcwalton removed that as part of his de-~str'ing, but I had already converted it to use StrBuf, so we may as well leave it in. Update: no longer, this is removed in a later commit.
I think it would be better to specify the length of the array as 0. It's also what GCC does in this situation, I think:
https://gcc.gnu.org/onlinedocs/gcc/Zero-Length.html
LLVM should know how to handle it (though I haven't tried).