Skip to content

Remove unused FCS support for compiling from ASTs #13242

@dsyme

Description

@dsyme
  1. FCS has an ability to take SyntaxTree as inputs instead of text. It was added by @eiriktsarpalis way back, but is to my knowledge entirely unused today - especially given the rate of churn in the specification of SyntaxTree as trivia has been added and other things cleaned up. I propose we remove this for simplicity's sake. At some later point we can take parsed inputs as input but there is no value in it at the moment.

  2. FCS also has the ability to create dynamic assemblies using Reflection.Emit, using the ilreflect.fs writer, including .NET Framework collectible assemblies. My proposal is we should also remove this, now that FSI has moved to using ilwrite.fs and ilreflect is now deprecated (and can't do things like emit debug symbols). Removing this would clean up a lot of entry point code.

Metadata

Metadata

Assignees

Type

No type

Projects

Status

Done

Relationships

None yet

Development

No branches or pull requests

Issue actions