src/Microsoft.Diagnostics.Runtime/src/Implementation/ClrmdType.cs (4 lines):
	- line 182: // TODO:  Handle typespec case.
	- line 284: // TODO move to ClrObject?
	- line 370: // TODO: remove
	- line 373: // TODO: remove


src/Microsoft.Diagnostics.Runtime/src/Implementation/ClrmdStringType.cs (4 lines):
	- line 155: // TODO:  Handle typespec case.
	- line 178: // TODO: remove
	- line 181: // TODO: remove
	- line 187: // TODO move to ClrObject?


eng/common/pipeline-logging-functions.ps1 (3 lines):
	- line 6: $script:loggingCommandEscapeMappings = @( # TODO: WHAT ABOUT "="? WHAT ABOUT "%"?
	- line 12: # TODO: BUG: Escape % ???
	- line 13: # TODO: Add test to verify don't need to escape "=".


src/Microsoft.Diagnostics.Runtime/src/Windows/HeapSegmentDataCache.cs (3 lines):
	- line 10: // TODO:  This code wasn't written to consider nullable.
	- line 177: // TODO: CurrentSize is constantly in flux, other threads are paging data in and out, so while I can read it in the below check there is no guarantee that by the time I get
	- line 250: // TODO: CurrentSize is constantly in flux, other threads are paging data in and out, so while I can read it in the below Where call there is no guarantee that by the time I get


src/Microsoft.Diagnostics.Runtime/src/Utilities/Command/Command.cs (2 lines):
	- line 221: // TODO : HACK we see to have a race in the async process stuff
	- line 299: // TODO (should we use WMI instead?)


src/Microsoft.Diagnostics.Runtime/src/Utilities/Command/CommandOptions.cs (2 lines):
	- line 230: // TODO add a capability to return a enumerator of output lines. (and/or maybe a delegate callback)
	- line 318: // TODO implement


src/Microsoft.Diagnostics.Runtime/src/Builders/RuntimeBuilder.cs (1 line):
	- line 1426: // TODO: Perf and testing


src/Microsoft.Diagnostics.Runtime/src/Windows/ArrayPoolBasedCacheEntry.cs (1 line):
	- line 10: // TODO:  This code wasn't written to consider nullable.


src/Microsoft.Diagnostics.Runtime/src/DacInterface/DacDataTargetWrapper.cs (1 line):
	- line 181: Debug.WriteLine($"TODO: Implement reading from module '{info.FileName}'");


src/Microsoft.Diagnostics.Runtime/src/Utilities/Platform/LinuxFunctions.cs (1 line):
	- line 118: // TODO:  This should be cleaned up to not read byte by byte in the future.  Leaving it here


src/Microsoft.Diagnostics.Runtime/src/Windows/AWEBasedCacheEntry.cs (1 line):
	- line 11: // TODO:  This code wasn't written to consider nullable.


src/Microsoft.Diagnostics.Runtime/src/Common/DACNameParser.cs (1 line):
	- line 519: // TODO: Is this a thing? The need to check for HasUnfulfilledGenericArgs? The code before this that propagates generic args only fixes a single generic type target


src/Microsoft.Diagnostics.Runtime/src/Windows/CacheEntryBase.cs (1 line):
	- line 9: // TODO:  This code wasn't written to consider nullable.


src/Microsoft.Diagnostics.Runtime/src/DataReaders/MacOS/MacOSProcessDataReader.cs (1 line):
	- line 358: internal const int PT_ATTACH = 10; // TODO: deprecated


src/Microsoft.Diagnostics.Runtime/src/Windows/CachePage.cs (1 line):
	- line 5: // TODO:  This code wasn't written to consider nullable.


src/Microsoft.Diagnostics.Runtime/src/Windows/AWEBasedCacheEntryFactory.cs (1 line):
	- line 27: // TODO: If I don't ensure that size is a multiple of the VA page size I leave chunks of unusable memory, then again allocating on a page boundary and not using it all is also


src/Microsoft.Diagnostics.Runtime/src/Implementation/ITypeHelpers.cs (1 line):
	- line 24: // TODO: Should not expose this:


src/Microsoft.Diagnostics.Runtime/src/Utilities/Platform/MacOSFunctions.cs (1 line):
	- line 92: // TODO:  This should be cleaned up to not read byte by byte in the future.  Leaving it here