Why is column information stripped from elixir's AST?

It already does in master! See the new :columns option in Code.string_to_quoted/2.

4 Likes

Actually I need column positions in the compile manifest, and from my explorations on the source, that needs a pretty serious refactor (down to the level of the erlang lexer maybe?). I have to see how disruptive such changes would be.

1 Like

I see. Yes, that’s not going to happen for now because the columns are not present on the code compiled by Elixir.

2 Likes

Hypothetically, if I ever find time to hack on Elixir’s compiler, would you be open to a PR?

2 Likes

The changes are actually quite small. My biggest concern is that it may break existing code. We took the first step in v1.6, let’s see how that will go. And we can take another step once v1.6 is out.

2 Likes