Record function param location

In its current form, this approach is pretty stupid. I feel like `params` should have a `loc.start` and `loc.stop` regardless of there being an empty parameter set. But I'm not sure how much we should be changing the data structures of this acorn fork.