Compiler: Ast_helper locations should not be optional

[ospencer]

It's too easy to accidentally forget to pass locations to parsetree nodes because they're optional.

(They're currently optional to make writing tests a little easier, but we no longer feel it's worth it.)