vsintegration/src/FSharp.ProjectSystem.Base/Project/ProjectNode.cs (14 lines):
	- line 489: // TODO cache an instance for perf; but be sure not to be stale (correctness)
	- line 2603: // TODO you get here if you have
	- line 2969: // TODO: Handle source control issues.
	- line 3516: // TODO: If source control is enabled check out the project file.
	- line 3690: // TODO : Revisit the VSADDFILEFLAGS here. Can it be a nested project?
	- line 3798: // REVIEW/TODO: shall we abandon accessing automation here and just look at MSBuild state?
	- line 3910: // TODO: possibly report in the error list that the the item is already contained in the project file similar to Language projects.
	- line 4501: // TODO: Refactor this code when we support user files
	- line 4508: // TODO: Refactor this code when we support user files
	- line 4559: // TODO: Refactor this code when we support user files
	- line 4656: // TODO: Refactor this code when we support user files
	- line 4753: // TODO: turn file watcher back on.
	- line 5994: // TODO: when adding support for User files, we need to update this method
	- line 6055: // TODO: when adding support for User files, we need to update this method


src/utils/reshapedreflection.fs (8 lines):
	- line 264: // TODO: is this an adequate replacement for MetadataToken
	- line 274: // TODO: is this an adequate replacement for MetadataToken
	- line 282: // TODO: is this an adequate replacement for MetadataToken
	- line 288: // TODO: is this an adequate replacement for MetadataToken
	- line 290: // TODO: is this an adequate replacement for MetadataToken
	- line 299: // TODO: is this an adequate replacement for MetadataToken
	- line 306: // TODO: is this an adequate replacement for MetadataToken
	- line 383: // TODO: is this an adequate replacement for MetadataToken


src/fsharp/AttributeChecking.fs (6 lines):
	- line 46: // TODO: typeof<..> in attribute values
	- line 73: // TODO: | TypeDefOfExpr g ty
	- line 130: // TODO: provided attributes
	- line 149: // TODO: provided attributes
	- line 162: // TODO: provided attributes
	- line 171: // TODO: provided attributes


vsintegration/src/FSharp.ProjectSystem.FSharp/MSBuildUtilities.fs (5 lines):
	- line 203: // TODO wildcards?
	- line 208: // TODO wildcards?
	- line 282: // TODO EnsureValid call below is extra work
	- line 289: // TODO EnsureValid call below is extra work
	- line 348: // TODO this logic is slightly broken, should probably use MSBuildProject.ItemIsVisible


vsintegration/src/FSharp.ProjectSystem.FSharp/Project.fs (5 lines):
	- line 316: //TODO the TypeProviderSecurityGlobals does not exists anymore, remove the initialization?
	- line 1363: // TODO: check whether this can be achieved in a less painful way
	- line 1908: let fullName = // TODO: 5571 tarcks replacing this with newMoniker.FullName
	- line 2714: // TODO: review - i'm not sure that node.Parent.FirstChild can ever == node because
	- line 2764: // TODO: review - i'm not sure that node.Parent.FirstChild can ever == node because


src/fsharp/ExtensionTyping.fs (5 lines):
	- line 552: static member CreateArray ctxt xs = match xs with null -> null | _ -> xs |> Array.map (ProvidedParameterInfo.Create ctxt)  // TODO null wrong?
	- line 997: // TODO: Property must be public etc.
	- line 1023: // TODO: Event must be public etc.
	- line 1032: | Some _  -> () // TODO: Constructors must be public etc.
	- line 1035: | Some _ -> () // TODO: Fields must be public, literals must have a value etc.


src/fsharp/service/IncrementalBuild.fsi (4 lines):
	- line 125: // TODO: make this an Eventually (which can be scheduled) or an Async (which can be cancelled)
	- line 131: // TODO: make this an Eventually (which can be scheduled) or an Async (which can be cancelled)
	- line 137: // TODO: make this an Eventually (which can be scheduled) or an Async (which can be cancelled)
	- line 143: // TODO: make this an Eventually (which can be scheduled) or an Async (which can be cancelled)


src/fsharp/FSharp.Core/Linq.fs (4 lines):
	- line 648: // TODO: amortize this computation based on f.Type
	- line 661: // TODO: amortize this computation based on f.Type
	- line 673: // TODO: amortize this computation based on f.Type
	- line 687: // TODO: amortize this computation based on f.Type


vsintegration/src/FSharp.ProjectSystem.Base/Project/IDEBuildLogger.cs (4 lines):
	- line 42: // TODO: Remove these constants when we have a version that supports getting the verbosity using automation.
	- line 44: // TODO: Re-enable this constants when we have a version that suppoerts getting the verbosity using automation.
	- line 689: // TODO: This should be replaced when we have a version that supports automation.
	- line 707: // TODO: Continue this code to get the Verbosity when we have a version that supports automation to get the Verbosity.


src/fsharp/IlxGen.fs (3 lines):
	- line 2467: // TODO - tupInfoRef
	- line 2468: argInfos |> List.iteri (fun i _ -> GenGetTupleField cenv cgbuf eenvinner (tupInfoRef (* TODO *),loce,tys,i,m) Continue);
	- line 3668: (*  TODO: Emit CompareExchange


src/fsharp/service/service.fs (3 lines):
	- line 444: match UntypedParseImpl.GetRangeOfExprLeftOfDot(endOfExprPos,Some(input)) with   // TODO we say "colAtEndOfNames" everywhere, but that's not really a good name ("foo  .  $" hit Ctrl-Space at $)
	- line 459: // TODO In theory I think we should never get to this code path; it would be nice to add an assert.
	- line 1290: // TODO uncomment this after `nameof` operator is implemented


vsintegration/src/FSharp.Editor/Completion/SignatureHelp.fs (3 lines):
	- line 81: // FSROSLYNTODO: FCS gives back line counts that are too large. Really, this shouldn't happen
	- line 168: // FSROSLYNTODO: compute the proper help text for parameters, c.f. AppendParameter in XmlDocumentation.fs
	- line 182: // FSROSLYNTODO: Do we need a cache like for completion?


src/buildtools/fsyacc/fsyaccast.fs (3 lines):
	- line 326: /// "spontaneous" items for the kernel. TODO: document this more w.r.t. the Dragon book.
	- line 336: /// "propagate" items for the kernel. TODO: document this more w.r.t. the Dragon book.
	- line 646: // TODO: this is very, very slow.


vsintegration/src/FSharp.LanguageService.Base/ViewFilter.cs (3 lines):
	- line 884: // TODO: This is failing, but it's not critical. We will enter into the if block
	- line 1184: pfFontData = 0; // TODO: Do whatever formatting we might want...
	- line 1188: pfFontData = 0; // TODO: Do whatever formatting we might want...


vsintegration/src/FSharp.LanguageService.Base/PatternMatcher/NormalizedTextSpanCollection.cs (2 lines):
	- line 431: // TODO: binary search
	- line 485: // TODO: binary search


src/fsharp/service/ServiceDeclarationLists.fs (2 lines):
	- line 86: // TODO: in this case ucinst is ignored - it gives the instantiation of the type parameters of
	- line 102: // TODO this code is similar to NicePrint.fs:formatParamDataToBuffer, refactor or figure out why different?


vsintegration/src/FSharp.LanguageService.Base/LanguageService.cs (2 lines):
	- line 1468: internal abstract ITrackingSpan[] GetParameterRanges();  // GetNoteworthyParamInfoLocations above is for unit testing; VS uses GetParameterRanges instead, to track changes as user types // TODO can we remove GetNoteworthyParamInfoLocations and move unit tests to GetParameterRanges?
	- line 1695: //TODO: reenable this!


src/fsharp/PatternMatchCompilation.fs (2 lines):
	- line 1053: // TODO: In the future we will want active patterns to be able to return struct-unions
	- line 1067: // TODO: In the future we will want active patterns to be able to return struct-unions


vsintegration/src/FSharp.ProjectSystem.Base/Project/FolderNode.cs (2 lines):
	- line 321: //TODO - this should not digest all exceptions.
	- line 359: //TODO - this should not digest all exceptions.


src/fsharp/symbols/Symbols.fs (2 lines):
	- line 2210: // TODO: there are several cases where we may need to report more interesting
	- line 2267: // TODO: the following don't currently return any interesting subtype


vsintegration/src/FSharp.ProjectSystem.Base/Project/AssemblyReferenceNode.cs (2 lines):
	- line 364: // TODO - Note that we don't currently support any logic for
	- line 540: // TODO - not accounting for case described below


src/absil/ilsupp.fs (2 lines):
	- line 671: check "Process.Start" p.ExitCode // TODO: really need to check against 0
	- line 757: // TODO: coalesce these two loops


vsintegration/src/FSharp.ProjectSystem.Base/Project/ProjectConfig.cs (2 lines):
	- line 126: // TODO rationalize this code with callers and ProjectNode.OnHandleConfigurationRelatedGlobalProperties, ProjectNode.TellMSBuildCurrentSolutionConfiguration, etc
	- line 1058: //TODO change to vsconstant when it is available in VsConstants. It should be guidCOMPlusNativeEng


vsintegration/src/FSharp.LanguageService.Base/Source.cs (2 lines):
	- line 647: // TODO: this doesn't work if the selection contains a mix of code and block comments
	- line 1238: // TODO maybe this gets into infinite loop of wasted work if something goes wrong


vsintegration/src/FSharp.Editor/Navigation/GoToDefinitionService.fs (2 lines):
	- line 155: // TODO: do we have any other terminating cases?
	- line 432: // FSROSLYNTODO: potentially display multiple results here


netci.groovy (2 lines):
	- line 22: // TODO: It should be possible to enable these configurations immediately subsequent to the PR containing this line
	- line 96: // TODO: set to false after tests are fully enabled


src/fsharp/symbols/Exprs.fs (2 lines):
	- line 360: // TODO: allow clients to see static initialization checks if they want to
	- line 1031: // TODO: this will not work for curried methods in F# classes.


vsintegration/src/FSharp.ProjectSystem.Base/Project/FileNode.cs (1 line):
	- line 870: // TODO does this do anything useful?  did it ever change in the first place?


src/fsharp/symbols/SymbolHelpers.fs (1 line):
	- line 1348: // TODO: check if this code can be implemented using xml-doc generation functionality


src/fsharp/service/ServiceUntypedParse.fs (1 line):
	- line 564: // TODO would be great to unify this with GetRangeOfExprLeftOfDot above, if possible, as they are similar


src/fsharp/service/IncrementalBuild.fs (1 line):
	- line 1163: member __.GetRawTypeForwarders() = mkILExportedTypes []  // TODO: cross-project references with type forwarders


vsintegration/src/FSharp.ProjectSystem.Base/Project/Automation/OAProjectItems.cs (1 line):
	- line 188: // TODO: VSADDITEMOP_LINKTOFILE


src/fsharp/fsi/fsi.fs (1 line):
	- line 92: // TODO: this dotnet/core polyfill can be removed when it surfaces in Type


vsintegration/src/FSharp.LanguageService.Base/CodeWindowManager.cs (1 line):
	- line 256: public void Done() { //TODO: use IDisposable pattern


vsintegration/src/FSharp.LanguageService/FSharpSource.fs (1 line):
	- line 269: // TODO: We should really create our own adornment layer.  It is possible (unlikely) that pre-existing layers may be re-ordered, or that


vsintegration/update-vsintegration.cmd (1 line):
	- line 316: REM TODO: this was already here (2017-09-28) and was already commented out, I think (AB) that these redirects aren't necessary anymore and can be permanently removed


vsintegration/src/FSharp.Editor/CodeFix/AddOpenCodeFixProvider.fs (1 line):
	- line 48: // TODO when fresh Roslyn NuGet packages are published, assign "Namespace" Tag to this CodeAction to show proper glyph.


vsintegration/src/FSharp.ProjectSystem.Base/Project/Automation/OANavigableProjectItems.cs (1 line):
	- line 115: // TODO:  Add OAProjectItems.Kind getter implementation


vsintegration/src/FSharp.ProjectSystem.Base/Project/ProjectReferenceNode.cs (1 line):
	- line 785: // TODO: Need to determine a generic way of finding browse library guids other than hardcoding


src/fsharp/ErrorLogger.fs (1 line):
	- line 286: // TODO: renable these asserts in the compiler service


src/fsharp/MethodCalls.fs (1 line):
	- line 717: // TODO: there  is a fair bit of duplication here with mk_il_minfo_call. We should be able to merge these


src/fsharp/CompileOps.fs (1 line):
	- line 2784: // TODO:  we have to get msbuild out of this


vsintegration/src/FSharp.Editor/Debugging/BreakpointResolutionService.fs (1 line):
	- line 56: // FSROSLYNTODO: enable placing breakpoints by when user supplies fully-qualified function names


src/fsharp/service/ServiceParamInfoLocations.fs (1 line):
	- line 80: | _ -> [""] // TODO type name for other cases, see also unit test named "ParameterInfo.LocationOfParams.AfterQuicklyTyping.CallConstructorViaLongId.Bug94333"


vsintegration/src/FSharp.ProjectSystem.Base/Project/Automation/VSProject/OAProjectReference.cs (1 line):
	- line 77: // TODO: Write the code that finds out the type of the output of the source project.


vsintegration/src/FSharp.LanguageService/XmlDocumentation.fs (1 line):
	- line 64: // TODO: bail out early if line limit is already hit


vsintegration/src/FSharp.Editor/LanguageService/LanguageService.fs (1 line):
	- line 520: // TODO: consider forceUpdate = false here.  forceUpdate=true may be causing repeated computation?


vsintegration/src/FSharp.Editor/Common/Pervasive.fs (1 line):
	- line 40: Some ()     // TODO: Should this be None?


src/fsharp/service/ServiceNavigation.fs (1 line):
	- line 690: // TODO: add specific case for record field?


vsintegration/src/FSharp.LanguageService/Intellisense.fs (1 line):
	- line 530: // TODO don't use QuickParse below, we have parse info available


vsintegration/src/FSharp.Editor/Debugging/LanguageDebugInfoService.fs (1 line):
	- line 49: // FSROSLYNTODO: This is used to get function names in breakpoint window. It should return fully qualified function name and line offset from the start of the function.


src/fsharp/fsc.fs (1 line):
	- line 795: //TODO compile error like c# compiler?


src/fsharp/QuotationTranslator.fs (1 line):
	- line 420: QP.mkTuple(tyR,argsR) // TODO: propagate to quotations


vsintegration/src/FSharp.ProjectSystem.FSharp/ProjectPrelude.fs (1 line):
	- line 65: // TODO: if we want to pick this location from registry?


scripts/dotnet-install.ps1 (1 line):
	- line 127: # TODO: Architecture and CLIArchitecture should be unified


vsintegration/src/FSharp.Editor/DocComments/XMLDocumentation.fs (1 line):
	- line 65: // TODO: bail out early if line limit is already hit


src/fsharp/FSharp.Build/FSharpEmbedResXSource.fs (1 line):
	- line 85: // TODO: When calling the `GetObject` version, parse the `type` attribute to discover the proper return type


src/buildtools/fslex/fslexast.fs (1 line):
	- line 259: // TODO: consider a better representation here.