chore(compiler): Make locs on anf_helper non optional

[spotandjake]

This pr makes locations non optional on anf_helper.

I think after this is merged we might want to open an issue to go through and evaluate the use of Location.dummy_loc throughout the compiler.

[phated]

Thanks for doing this @spotandjake! I wonder if it is actually better to use this diff as a way for @ospencer and @peblair to inventory the usage of dummy_loc 🤔

[spotandjake]

Thanks for doing this @spotandjake! I wonder if it is actually better to use this diff as a way for @ospencer and @peblair to inventory the usage of dummy_loc 🤔

There are other cases of it being used in the compiler that I think might be worth inventorying as well.

[ospencer]

Did inventory of the usage of dummy_loc here. Caught a couple, though by nature of what's happening in the linearization/matchcomp phase we're generating a number of expressions that don't have corresponding locations in the source tree. In the future we may decide to handle those differently (maybe just trying to attribute it to the closest node or something).

Inventory complete