Change natural type of UTF-8 string literals to `ReadOnlySpan<byte>` and null terminate the underlying blob. #61532

AlekseyTs · 2022-05-26T17:08:42Z

https://github.com/dotnet/csharplang/blob/main/meetings/2022/LDM-2022-04-18.md#natural-type-of-utf8-literals
https://github.com/dotnet/csharplang/blob/main/meetings/2022/LDM-2022-04-18.md#should-utf8-literals-be-null-terminated

Related to #61517
Closes #60644

Test plan #58848

…and null terminate the underlying blob. https://github.com/dotnet/csharplang/blob/main/meetings/2022/LDM-2022-04-18.md#natural-type-of-utf8-literals https://github.com/dotnet/csharplang/blob/main/meetings/2022/LDM-2022-04-18.md#should-utf8-literals-be-null-terminated Related to dotnet#61517

jcouv · 2022-05-26T17:36:43Z

src/Compilers/CSharp/Portable/Binder/Binder.ValueChecks.cs

@@ -2613,6 +2613,7 @@ internal static uint GetValEscape(BoundExpression expr, uint scopeOfTheContainin
                case BoundKind.DefaultExpression:
                case BoundKind.Parameter:
                case BoundKind.ThisReference:
+                case BoundKind.UTF8String:


Usually, GetValEscape and GetRefEscape need to be updated in pair. Does GetRefEscape need to be updated too? #Resolved

Does GetRefEscape need to be updated too?

Based on the comment in the method, it is supposed to handle "all the possible cases for anything that is not a strict RValue". I think the literal is a strict RValue.

jcouv · 2022-05-26T18:02:01Z

src/EditorFeatures/CSharpTest/ConvertNamespace/ConvertToFileScopedNamespaceAnalyzerTests.cs

@@ -737,14 +737,16 @@ class C
    {
        void M()
        {
-            System.Console.WriteLine(""""""


nit: Just curious, do we know if the BCL is planning to add some Console APIs for ROS<byte>? #Resolved

Console's APIs are in terms of text, e.g. string. Adding bytes-based APIs would require knowing what encoding to use to convert those bytes to text. It may not be the same encoding of the underlying stream. If you want to work in terms of bytes, you can always get a Stream and write to that, at which point you're saying you know your bytes are appropriate for the underlying encoding, e.g.

using Stream s = Console.OpenStandardOutput(); s.Write("hello world"u8);

jcouv · 2022-05-26T18:09:10Z

src/Compilers/CSharp/Portable/CodeGen/EmitArrayInitializer.cs

@@ -410,6 +411,39 @@ private bool TryEmitReadonlySpanAsBlobWrapper(NamedTypeSymbol spanType, BoundExp
                return false;
            }

+            if (start is null != length is null)
+            {
+                return false;


nit: Consider asserting (unreachable at the moment) #Resolved

jcouv · 2022-05-26T18:24:48Z

src/Compilers/CSharp/Test/Semantic/Semantics/Utf8StringsLiteralsTests.cs

@@ -2018,7 +1884,7 @@ static void Main()
 }
 ";
            var comp = CreateCompilation(source, targetFramework: TargetFramework.NetCoreApp, options: TestOptions.DebugExe);
-            CompileAndVerify(comp, expectedOutput: @"Span").VerifyDiagnostics();
+            CompileAndVerify(comp, expectedOutput: @"ReadOnlySpan", verify: Verification.Fails).VerifyDiagnostics();


Why was IL verification succeeding with Span but now fails with ReadOnlySpan overload and new natural type? #Resolved

Why was IL verification succeeding with Span but now fails with ReadOnlySpan overload and new natural type?

The old code was creating a byte array and then using conversion operator to convert it to Span. No special constructors were used.

jcouv · 2022-05-26T18:30:00Z

src/Compilers/CSharp/Test/Semantic/Semantics/Utf8StringsLiteralsTests.cs

-
-            var symbolInfo = model.GetSymbolInfo(node);
-            Assert.Equal("System.Span<System.Byte> System.Span<System.Byte>.op_Implicit(System.Byte[]? array)", symbolInfo.Symbol.ToTestDisplayString());
+            Assert.Null(symbolInfo.Symbol);


Consider also verifying diagnostics #WontFix

Consider also verifying diagnostics

I think other tests provide enough coverage for diagnostics.

jcouv

Done with review pass (iteration 1)

AlekseyTs · 2022-05-26T20:00:08Z

@RikkiGibson, @cston, @dotnet/roslyn-compiler For the second review.

In reply to: 1138956553

davidwengier

Thank you for disabling the analyzer

AlekseyTs · 2022-05-27T16:59:16Z

@RikkiGibson, @cston, @dotnet/roslyn-compiler For the second review.

In reply to: 1139801088

RikkiGibson · 2022-05-27T18:16:35Z

src/Compilers/CSharp/Portable/Lowering/LocalRewriter/LocalRewriter_Conversion.cs

+
+            return result;
+
+            BoundExpression createUTF8ByteRepresentation(SyntaxNode resultSyntax, SyntaxNode valueSyntax, string value, ArrayTypeSymbol byteArray, out int length)


nit: should 'resultSyntax' and 'valueSyntax' be replaced with a single argument here? #Resolved

RikkiGibson · 2022-05-27T18:33:32Z

src/Compilers/CSharp/Test/Semantic/Semantics/Utf8StringsLiteralsTests.cs

-  IL_0016:  ret
+  IL_0011:  ldc.i4.0
+  IL_0012:  ldc.i4.3
+  IL_0013:  newobj     ""System.ReadOnlySpan<byte>..ctor(byte[], int, int)""


I am confused about when we do which of the following in various scenarios:

create an array, then create a span referencing it (which we do here)

create a span referencing a blob (which we do in some of the other tests)

I expected that for UTF-8 string literals, we would only need to do (2) and not (1). Could you please explain? #Resolved

I am confused about when we do which of the following in various scenarios:

create an array, then create a span referencing it (which we do here)

create a span referencing a blob (which we do in some of the other tests)

I expected that for UTF-8 string literals, we would only need to do (2) and not (1). >Could you please explain?

Utilizing a blob is an optimization strategy that is applied by emit layer. It is applied conditionally, including some conditions that aren't related to specific array data.
This can be observed in TryEmitReadonlySpanAsBlobWrapper helper. For example:

Is _module.SupportsPrivateImplClass true

Is WellKnownMember.System_ReadOnlySpan_T__ctor_Pointer constructor available

In the emit layer we are trying to optimize usage of other (non-pointer) constructors and some conversions. The WellKnownMember.System_ReadOnlySpan_T__ctor_Pointer is not used in any other compiler layers.

In lowering, we are always using a constructor that takes array, start and length. Emit layer might or might not optimize it with a blob. This specific test covers scenario when WellKnownMember.System_ReadOnlySpan_T__ctor_Pointer is unavailable. Note comp.MakeMemberMissing(WellKnownMember.System_ReadOnlySpan_T__ctor_Pointer); above. This disables the optimization and the usage of the regular constructor survives.

RikkiGibson

LGTM, thanks!

…ures/required-members * upstream/main: Change natural type of UTF-8 string literals to `ReadOnlySpan<byte>` and null terminate the underlying blob. (dotnet#61532)

…and null terminate the underlying blob. (#61532) (#61570) https://github.com/dotnet/csharplang/blob/main/meetings/2022/LDM-2022-04-18.md#natural-type-of-utf8-literals https://github.com/dotnet/csharplang/blob/main/meetings/2022/LDM-2022-04-18.md#should-utf8-literals-be-null-terminated Related to #61517 Closes #60644 Co-authored-by: AlekseyTs <[email protected]>

…ures/semi-auto-props * upstream/main: (111 commits) Add unit test project IVT to ExternalAccess.AspNetCore PR feedback Use VS2022 for PR Validation builds Bind native integers in cref (#61431) fix assumption of length Change natural type of UTF-8 string literals to `ReadOnlySpan<byte>` and null terminate the underlying blob. (#61532) IDE Support for Required Members (#61440) Final prototype comments and top level statements local adjustments (#61551) Add IsDefault Add some APIs on AspNetCoreVirtualCharSequence Skip timing test (#61222) Prepare VB iterators for EnC support (#61488) Improve normalization to match idiomatic patterns for nested usings and fixed statements. (#61533) Update unit tests Don't throw in logging when the document path contains curly braces (#61524) Fix AbstractLanguageService constructor (#61513) Remove Utf8StringLiteral conversion (#61481) Use feature attribute Pull token out Make async ...

@jkoritzinsky

* Avoid caching RelativeIndentationData.effectiveBaseToken * Relax assertion in SyntheticBoundNodeFactory.Convert (#61287) * Add missing node state transition It's valid for an input to be modified causing a downstream input to be removed (for example a syntax tree can change what is in it, leading to the downstream node not generating something). //cc @jkoritzinsky in case I'm missing something obvious. * Simplify logic, more tests, rework tests * Fix several LSP completion kind mappings (#61243) * Fix several LSP completion kind mappings * Extension method * Fix issue where we were getting a raw-string in a skipped token, causing a crash * Add test * Add an UWP OptProf test for IDE * Disable smart copy/paste when line-copy is involved. * Update src/EditorFeatures/CSharp/StringCopyPaste/StringCopyPasteCommandHandler.cs * Add additional sanity check * Fix * Add pointer for `AnalysisLevel` to warning waves doc (#61196) * Remove parameter null-checking from the Language Feature Status list (#61302) * Switch to GetRequiredService * Simpler approach * [EE] Implement IDkmClrFullNameProvider2 in Roslyn's ResultProvider Formatter. (#60522) * Implement IDkmClrFullNameProvider2 in Roslyn's ResultProvider Formatter. Issue: - Debugger added IDkmClrFullNameProvider2 API with https://devdiv.visualstudio.com/DevDiv/_git/Concord/pullrequest/301518 - It is currently temporarily implement in Concord, needs to be implemented in Roslyn so implementation can be removed from Concord. Changes: 1. Formatter: - Implement IDkmClrFullNameProvider2. It has 2 methods, one to format local names and the other given field metadata. - Currently only implemented for C#. I'm not that familiar with VB and the GeneratedNames stuff in VB needs some splitting and moving around to get working. 2. Unit tests: - Add unit tests for the common cases of hoisted locals, synthesized locals, etc. 3. Versions.props: Update MicrosoftVSSDKVSDConfigToolVersion to a newer version which recognizes IDkmClrFullNameProvider2. * PR feedback - fix casing of MetadataImport * PR feedback - move GetOriginalLocalVariableName, GetOriginalFieldName back to CSharpFormatter Co-authored-by: Ramkumar Ramesh <[email protected]> * PR feedback * Fix and/or completion after parenthesized pattern * [LSP] Disable GoToDef/GoToImpl integration tests (#61190) * Added syntax context flag * Keywords c# * Keep leadin trivia inside Main method if it is more likely to be a statement comment rather than a file header when converting to 'Program.Main' style program * Symbols * Snippets C# * Remove set accessor of new SyntaxContext property * Remove PROTOTYPE comments (#61322) * Added assertions, comments, and refactored for clarity * Change VB language version Roslyn.sln uses to "latest" (#61313) To allow us to use the latest VB features, such as setting init-only properties. * PR feedback * Remove document options provider (#61228) * Remove IDocumentOptionsProvider * Fold DocumentSpecificOptionSet into DocumentOptionSet * Use an explicit option to control frozen-partial semantics in inheritance margin * Break into separate methods * Fix null ref (#61342) * Simplify internal types search * Removed unintentional WorkItem's * Make static * PR feedback * [LSP] Support LSP services associated with LSP server instances (with lifetimes that match). (#61266) * Add support for exporting services that are created for each server instance. * Use the correct span to rename after invoking extract-method manually. * Move Spellcheck capabilities to be activated in all scenarios (#61366) * move spellcheck capability to always activated server * Rename type * rename fields * [LSP] Add JSON semantic token classifications (#61231) * Make async * Test fallout * Unify nint and IntPtr (#60913) * restore file * Minor simplification to rename code * Remaining fallback options (#60888) * Add missing fallbacks * Fallback options from ILegacyGlobalOptionsWorkspaceService * Pass options to CodeCleaner APIs. * Fallback options from ILegacyGlobalOptionsWorkspaceService 2 * Fallback in tests * CodeModel * Remove CodeActionOptions.Default * Remove dependency on IGlobalOptionService from inline hints service * Remove obsolete VS UnitTesting APIs. * Remote dependency on IGlobalOption service from RemoteProcessTelemetryService * Remove ExportGlobalOptionProviderAttribute * Remove PythiaOptions * Remove DiagnosticOptions from solution snapshot * Access options via AnalyzerOptionsProvider * Split ISyntaxFormatting.cs * Simplify initializers * Fix * Move AddImportPlacementOptions to a separate file in compiler extensions * Move option providers to workspace extensions * Move CodeCleanupOptions and IdeAnalyzerOptions to workspace extensions * Layering * Replace legacy GetOptions with AnalyzerOptionsProvider; add missing options * Parameter rename, comment * Add LineFormattingOptionsProviders * CodeFixOptionsProvider, include CodeStyleOptions in CodeActionOptions, include LineFormattingOptions in ExtractMethodGenerationOptions * DocumentFormattingOptions * Move a couple of options from IdeCodeStyle to SyntaxFormatting to make them available to new document formatter * Generalize using placement option in AddImportPlacementOptions * Move PreferParameterNullChecking and AllowEmbeddedStatementsOnSameLine to CSharpSimplifierOptions * Move CodeGen options to compiler extensions * UseExpressionBody * Eliminate more calls to Document.GetOptionsAsync * Cleanup DocumentationCommentOptions * Line formatting options * DefaultConditionalExpressionWrappingLength * insert_final_newline * Add PreferThrowExpression to simplifier options * Add AddNullChecksToConstructorsGeneratedFromMembers to CodeGenOptions * GenerateEqualsAndGetHashCodeFromMembersOptions * IImplementInterfaceService * AddParameterCheckCodeRefactoringProvider * ReplaceMethodWithPropertyService * NamingStylePreferences * Eliminate legacy option helpers * Fix up ExtractMethod options * Remove SyntaxFormattingOptions ctors * Replace extra helpers with CodeFixOptionsProvider * PreferUtf8StringLiterals * RazorLineFormattingOptionsStorage * Remove usage of Document.GetOptionsAsync - 1 * Remove internal usage of DocumentOptionSet and Document.GetOptionsAsync * Simplify and unify option definition patterns * Fixes and pattern unification * Serialization and equality * Simplify * Rename * Fixes * CompletionOption fixes * Feedback * Single switch * Add frozen delegate tests * Add extract method test * Add workitem * Fix * Add support for CompilerFeatureRequiredAttribute (#61113) Adds support for decoding and reporting errors when `CompilerFeatureRequiredAttribute` is encountered on metadata type symbols. We also block applying the attribute by hand in both C# and VB. * Delegate keyword tests * Verifying interpolation escaping of curlies in content (#61387) * Verify classification on var pattern (#61376) * Add lambda parameters in scope in nameof using proper binder (#61382) * Parse `unchecked` gracefully in operators (#61309) * Update src/Features/Core/Portable/InheritanceMargin/AbstractInheritanceMarginService_Helpers.cs * Update src/Features/Core/Portable/InheritanceMargin/AbstractInheritanceMarginService_Helpers.cs * Use AspNetCoreKey to external access assembly * Fix typo (#61380) * Add new collapsing option for metadata files that contain source (#61205) * More correctly respect background analysis scope (#61392) * Fix function id (#61400) * Wait for async operations to complete before proceeding * Update SDK to .NET 7 Preview 4 * Avoid logging work when no logger is specified * Emit CompilerFeatureRequired for ref structs when present. * Support emitting CompilerFeatureRequiredAttribute for contructors of types with required members. * [LSP] Small cleanup for pull diagnostics logging (#61417) * Small logging cleanup on pull diagnostics code * Update tests to account for lsp diagnostics throwing when mismatch in diagnostic mode * Address feedback from numeric IntPtr feature review (#61418) * Fix binding for checkbox text in rename dialogs (#61430) Previously text was bound to properties on the control type using x:Name. This restores that * Fix build * Restrict IsGenericConstraintContext for C# * More instrumentation for ReferenceCachingCS (#61402) * Do not filter snippets * Reverted delegate completion * Allow source link, embedded or decompiled source in Peek Definition (#61427) * Update status for DIM and numeric IntPtr (#61464) * Lazily produce semantic models in source generators * Remove unnecessary finalizer state handling from MethodToStateMachineRewriter (#61409) * Update src/Tools/IdeCoreBenchmarks/IncrementalSourceGeneratorBenchmarks.cs * Update src/Tools/IdeCoreBenchmarks/IncrementalSourceGeneratorBenchmarks.cs * Update src/Tools/IdeCoreBenchmarks/IncrementalSourceGeneratorBenchmarks.cs * lint * Address prototype comments (#61436) * Give a warning when obsolete is applied to a required member and the containing context is not obsolete, or all constructors are not obsolete/setsrequiredmembers. * Restore nullable constructor warnings for constructors with `SetsRequiredMembersAttribute`. * Remove prototype comments. * Add tests and extra state * Extract checking for generic constraint context to extension method & minor refactoring * Update test comment * Generate single OptProf config for compiler vsix Currently OptProf can't support profiling for multiple flavors of vsix, and Optprof test only runs on X64. Multiple configs with same profiling binary are causing the tests to fail. * Reduce release/64 limit for EndToEndTests.Constraints (#61480) * Reduce release/64 limit for EndToEndTests.Constraints * Lower bar more * Lower bar more * Remove parameter nullchecking feature (#61397) * Fix null ref for JS files (#61472) * Add file paths to interactive buffers and documents to support LSP requests (#61441) * Add file paths to interactive buffers and documents to support LSP requests * Switch to false returning predicate * more feedback * mroe feedback * Reword comment * Log additional information from CopyRefAssembly (#61384) * update versioning to use languageserver.client.implementation * Require VS 17.0 in signed build. * Bump LSP protocol version (#61494) * Bump protocol version * React to breaking changes in foldingrangekind * Disable inheritance margin for interactive documents (#61476) * fix * Improvements to the background compiler component * add docs * Simplify * docs * Check token * Simplify * Simplify * Rename enum field * Lifted relational operator implies operands non-null when true (#61403) * Use ImmutableArray instead of IEnumerable parameters * Use assignabiilty instead of subclass test in extension loader * Fix test * Fix test * Simplify rename implementation * Rename * Unify all end operations that rename performs * restore code * Remove unnecessary code' * Unify error handling in rename * Add comment * message severities * Restore * Simplify * Bring main-vs-deps back (#61514) * Add main-vs-deps back * Update eng/config/PublishData.json Co-authored-by: Joey Robichaud <[email protected]> Co-authored-by: Joey Robichaud <[email protected]> * Simplify LSP reference update * Adjust conversion from nuint to float/double (#61345) * Add embedded classification for field initializers * Add support for properties * Update src/EditorFeatures/Core/InlineRename/InlineRenameSession.cs * Update src/EditorFeatures/Core/RenameTracking/RenameTrackingTaggerProvider.RenameTrackingCommitter.cs * Update src/EditorFeatures/Core/RenameTracking/RenameTrackingTaggerProvider.RenameTrackingCommitter.cs * Fix setup authoring bug (#61508) * Expose VirtualChars to asp.net (through EA) to facilitate route classification * add docs * Add member * NRT * Make async * Make async * Pull token out * Use feature attribute * Remove Utf8StringLiteral conversion (#61481) https://github.com/dotnet/csharplang/blob/main/meetings/2022/LDM-2022-04-18.md#target-typing-a-regular-string-literal-to-utf8-types * Fix AbstractLanguageService constructor (#61513) * Fix AbstractLanguageService constructor * Fix formatting * Don't throw in logging when the document path contains curly braces (#61524) * Update unit tests * Improve normalization to match idiomatic patterns for nested usings and fixed statements. (#61533) * Better syntax normalization for fixed/using statements * Add tests * Prepare VB iterators for EnC support (#61488) * Remove unused parameters * Separate iterator finalizer states from resumable states. * Remove unused * Skip timing test (#61222) * Skip test * Add some APIs on AspNetCoreVirtualCharSequence * Add IsDefault * Final prototype comments and top level statements local adjustments (#61551) Clean up the last of the prototype comments and adjust the parsing of locals named required in top level statements. * IDE Support for Required Members (#61440) * Add required keyword recommender. * Add SyntaxNormalizer test. * Code generation support. * Add SymbolDisplay * F1 help service and test fix. * Add order modifier tests and update. * Change natural type of UTF-8 string literals to `ReadOnlySpan<byte>` and null terminate the underlying blob. (#61532) https://github.com/dotnet/csharplang/blob/main/meetings/2022/LDM-2022-04-18.md#natural-type-of-utf8-literals https://github.com/dotnet/csharplang/blob/main/meetings/2022/LDM-2022-04-18.md#should-utf8-literals-be-null-terminated Related to #61517 Closes #60644 * fix assumption of length * Bind native integers in cref (#61431) * Use VS2022 for PR Validation builds * PR feedback * Add unit test project IVT to ExternalAccess.AspNetCore * Fix generation location when generating across files * Add test * Relax check * EnC: Allow adding/removing await expressions and yield statements (#61356) * Implements support for adding and removing await/yield return in C# async, iterator and async iterator methods. * Fix syntax node associated with BoundTryStatement created from using syntax * Update required members status (#61602) * Implements support for adding and removing await/yield return in C# in the IDE (#61521) Co-authored-by: Sam Harwell <[email protected]> Co-authored-by: Julien Couvreur <[email protected]> Co-authored-by: Chris Sienkiewicz <[email protected]> Co-authored-by: DoctorKrolic <[email protected]> Co-authored-by: Gen Lu <[email protected]> Co-authored-by: DoctorKrolic <[email protected]> Co-authored-by: Cyrus Najmabadi <[email protected]> Co-authored-by: [email protected] <[email protected]> Co-authored-by: CyrusNajmabadi <[email protected]> Co-authored-by: dotnet bot <[email protected]> Co-authored-by: AlFas <[email protected]> Co-authored-by: Ramkumar Ramesh <[email protected]> Co-authored-by: Ramkumar Ramesh <[email protected]> Co-authored-by: AlekseyTs <[email protected]> Co-authored-by: Allison Chou <[email protected]> Co-authored-by: Tomáš Matoušek <[email protected]> Co-authored-by: David Wengier <[email protected]> Co-authored-by: David Barbet <[email protected]> Co-authored-by: Fred Silberberg <[email protected]> Co-authored-by: James Newton-King <[email protected]> Co-authored-by: Weihan Li <[email protected]> Co-authored-by: Joey Robichaud <[email protected]> Co-authored-by: Julien Couvreur <[email protected]> Co-authored-by: Andrew Hall <[email protected]> Co-authored-by: Manish Vasani <[email protected]> Co-authored-by: Jared Parsons <[email protected]> Co-authored-by: Joey Robichaud <[email protected]> Co-authored-by: Rikki Gibson <[email protected]> Co-authored-by: akhera99 <[email protected]> Co-authored-by: Ankita Khera <[email protected]>

AlekseyTs added Area-Compilers New Feature - Utf8StringLiterals labels May 26, 2022

AlekseyTs requested review from davidwengier, RikkiGibson and cston May 26, 2022 17:08

AlekseyTs requested review from a team as code owners May 26, 2022 17:08

jcouv reviewed May 26, 2022

View reviewed changes

jcouv self-assigned this May 26, 2022

runfoapp bot mentioned this pull request May 26, 2022

roslyn.visualstudio.newintegrationtests.csharp.csharperrorlistnetcore.errorsafterclosingfile is flaky #61368

Open

PR feedback and test adjustments

6abcd69

jcouv approved these changes May 26, 2022

View reviewed changes

davidwengier approved these changes May 26, 2022

View reviewed changes

RikkiGibson self-assigned this May 27, 2022

RikkiGibson reviewed May 27, 2022

View reviewed changes

Merge two parameters

848008d

RikkiGibson approved these changes May 27, 2022

View reviewed changes

AlekseyTs enabled auto-merge (squash) May 27, 2022 19:39

AlekseyTs merged commit 54586cd into dotnet:main May 27, 2022

ghost added this to the Next milestone May 27, 2022

wtgodbe mentioned this pull request May 31, 2022

Cleanup u8 string literal workarounds dotnet/aspnetcore#41455

Closed

Cosifne modified the milestones: Next, 17.3 P2 May 31, 2022

jcouv mentioned this pull request Jun 2, 2022

Use RuntimeHelpers.CreateSpan for array initialization with relevant primitives #61414

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Change natural type of UTF-8 string literals to `ReadOnlySpan<byte>` and null terminate the underlying blob. #61532

Change natural type of UTF-8 string literals to `ReadOnlySpan<byte>` and null terminate the underlying blob. #61532

AlekseyTs commented May 26, 2022 •

edited by jcouv

Loading

jcouv May 26, 2022 •

edited by AlekseyTs

Loading

AlekseyTs May 26, 2022

jcouv May 26, 2022 •

edited by AlekseyTs

Loading

stephentoub May 26, 2022

jcouv May 26, 2022 •

edited by AlekseyTs

Loading

jcouv May 26, 2022 •

edited by AlekseyTs

Loading

AlekseyTs May 26, 2022

jcouv May 26, 2022 •

edited by AlekseyTs

Loading

AlekseyTs May 26, 2022

jcouv left a comment

AlekseyTs commented May 26, 2022 •

edited

Loading

davidwengier left a comment

AlekseyTs commented May 27, 2022 •

edited

Loading

RikkiGibson May 27, 2022 •

edited by AlekseyTs

Loading

RikkiGibson May 27, 2022 •

edited by AlekseyTs

Loading

AlekseyTs May 27, 2022 •

edited

Loading

RikkiGibson left a comment


		return result;

		BoundExpression createUTF8ByteRepresentation(SyntaxNode resultSyntax, SyntaxNode valueSyntax, string value, ArrayTypeSymbol byteArray, out int length)

Change natural type of UTF-8 string literals to ReadOnlySpan<byte> and null terminate the underlying blob. #61532

Change natural type of UTF-8 string literals to ReadOnlySpan<byte> and null terminate the underlying blob. #61532

Conversation

AlekseyTs commented May 26, 2022 • edited by jcouv Loading

jcouv May 26, 2022 • edited by AlekseyTs Loading

Choose a reason for hiding this comment

AlekseyTs May 26, 2022

Choose a reason for hiding this comment

jcouv May 26, 2022 • edited by AlekseyTs Loading

Choose a reason for hiding this comment

stephentoub May 26, 2022

Choose a reason for hiding this comment

jcouv May 26, 2022 • edited by AlekseyTs Loading

Choose a reason for hiding this comment

jcouv May 26, 2022 • edited by AlekseyTs Loading

Choose a reason for hiding this comment

AlekseyTs May 26, 2022

Choose a reason for hiding this comment

jcouv May 26, 2022 • edited by AlekseyTs Loading

Choose a reason for hiding this comment

AlekseyTs May 26, 2022

Choose a reason for hiding this comment

jcouv left a comment

Choose a reason for hiding this comment

AlekseyTs commented May 26, 2022 • edited Loading

davidwengier left a comment

Choose a reason for hiding this comment

AlekseyTs commented May 27, 2022 • edited Loading

RikkiGibson May 27, 2022 • edited by AlekseyTs Loading

Choose a reason for hiding this comment

RikkiGibson May 27, 2022 • edited by AlekseyTs Loading

Choose a reason for hiding this comment

AlekseyTs May 27, 2022 • edited Loading

Choose a reason for hiding this comment

RikkiGibson left a comment

Choose a reason for hiding this comment

Change natural type of UTF-8 string literals to `ReadOnlySpan<byte>` and null terminate the underlying blob. #61532

Change natural type of UTF-8 string literals to `ReadOnlySpan<byte>` and null terminate the underlying blob. #61532

AlekseyTs commented May 26, 2022 •

edited by jcouv

Loading

jcouv May 26, 2022 •

edited by AlekseyTs

Loading

jcouv May 26, 2022 •

edited by AlekseyTs

Loading

jcouv May 26, 2022 •

edited by AlekseyTs

Loading

jcouv May 26, 2022 •

edited by AlekseyTs

Loading

jcouv May 26, 2022 •

edited by AlekseyTs

Loading

AlekseyTs commented May 26, 2022 •

edited

Loading

AlekseyTs commented May 27, 2022 •

edited

Loading

RikkiGibson May 27, 2022 •

edited by AlekseyTs

Loading

RikkiGibson May 27, 2022 •

edited by AlekseyTs

Loading

AlekseyTs May 27, 2022 •

edited

Loading