Fix "harmless" bug in ZapImport - use AppendByte to emit fixup type (#21621)

During my work on the CPAOT compiler I found out that in the past
I had believed we should use the compressed uint signature encoding
for emitting fixup types. This place may have contributed to my
mistake. I have audited all places in CoreCLR where signatures are
decoded and in all cases the fixup type is assumed to be a byte,
not a signature-compressed uint.

As I said, the bug is harmless (after all, Crossgen works) because
the enum code of READYTORUN_FIXUP_Helper is less than 128 and for
values 0-127 the compressed uint encoding is identical with the
byte value; I however believe it's useful to make this change
nonetheless for the sake of code clarity and to help avoid future
confusions similar to mine.

Fix jit warning in crossgen desktop build.

Don't morph volatile IND(ADDR(LCL_VAR)) (#20843)

Besides the fact that volatile indirections aren't supposed to be removed doing so in this case results in incorrect code being generated.

The GT_IND node has GTF_DONT_CSE set and this gets copied to the GT_LCL_VAR node. Later this prevents the insertion of a normalization cast because GTF_DONT_CSE on a GT_LCL_VAR node is assumed to mean that the variable address is being taken.

Merge pull request #21589 from CarolEidt/Fix21546

Transform SIMD8 to FIELD_LIST if promoted

Update CoreClr, CoreFx to preview-27220-03, preview.18619.1, respectively (#21592)

Freebsd symbols3 (#21607)

* enable symbols on freebsd once again

Replace more of JitHelpers with S.R.CS.Unsafe (#21598)

- Replace JitHelpers.UnsafeCastToStackPtr with Unsafe.AsPointer
- Delete PinningHelper that was duplicate of RawData helper class

Update dependency files (#21580)

* Update dependencies from https://github.com/dotnet/arcade build 844

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.18617.7
- Microsoft.DotNet.Helix.Sdk - 1.0.0-beta.18617.7

* Update dependencies from https://github.com/dotnet/arcade build 885

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.18618.7
- Microsoft.DotNet.Helix.Sdk - 1.0.0-beta.18618.7

* Update dependencies from https://github.com/dotnet/arcade build 908

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.18619.4
- Microsoft.DotNet.Helix.Sdk - 1.0.0-beta.18619.4

Fix jit warning. (#21600)

Add build timeout (#21591)

This should fix official build failures that are timing out because
the jobs are waiting to obtain an exclusive lock on the blob
feed.

Update CoreClr, CoreSetup to preview-27218-02, preview-27218-01, respectively (#21558)

Transform SIMD8 to FIELD_LIST if promoted

Fix #21546

Update named mutex PAL tests to include case of file locks being inherited by child process (#21496)

Included case fixed by https://github.com/dotnet/coreclr/pull/21458. Fixes https://github.com/dotnet/coreclr/issues/21494.

Add support for parsing field signatures (#21573)

This change adds basic field signature parsing support to R2RDump
and it improves parsing of two fixup types (FIELD_ADDRESS and
CCTOR_TRIGGER).

Thanks

Tomas

Merge pull request #21567 from CarolEidt/Issue20657

Test for #20657 SIMD byref bug

Merge pull request #21571 from BruceForstall/DisableArm32Test

Disable GitHub_19361 for Linux arm32

Fix stack walking and reporting of default interface methods (#21525)

Default interface methods in their unresolved state don't have a generic context. The generic context is only added once the method is resolved to its implementation.

Add .NET Core version to EventLog (#21566)

Generate ARM CFIs in prolog (#21505)

Fixed suppressed XML documentation errors (dotnet/corefx#34020)

* Fixed XML documentation tags for CoreFx.Private.TestUtilities

* Fixed XML documentation tags for System.Diagnostics.Process

* Fixed XML documentation tags for System.ComponentModel.Composition

* Fixed XML documentation tags for System.ComponentModel.Primitives

* Fixed XML documentation tags for System.ComponentModel.TypeConverter

* Fixed XML documentation tags for System.Data.Common

* Fixed XML documentation tags for System.Data.SqlClient

* Fixed XML documentation tags for System.Diagnostics.PerformanceCounter

* Fixed XML documentation tags for System.Diagnostics.StackTrace

* Fixed XML documentation for System.Diagnostics.TraceSource

* Fixed XML documentation for System.IO.Compression.ZipFile

* Fixed XML documentation for System.IO.Compression

* Fixed XMl documentation reference according to review notes

* Fixed XML documentation tags for System.IO.MemoryMappedFiles

* Fixed XML documentation tags for System.IO.Pipes

* Fixed XML documentation tag for OSX interop in System.Diagnostics.Process

* Fixed XML documentation tags for CoreFx.Private.TestUtilities for Unix

* Fixed XML documentation tags for System.Linq.Expressions

* Fixed XML documentation for System.Linq.Parallel

* Fixed XML documentationt ags for System.Linq

* Fixed XML documentation tags for System.Management

* Fixed XMl documentation tags for System.Net.Http (left CS1573)

* Fixed XMl documentation tags for System.Net.Mail

* Fixed XML documentation tags for System.Net.NameResolution

* Fixed XML documentation tags for System.Net.Primitives - added T prefix for Socket as Primitives do not reference System.Net.Sockets

* Fixed XML documentation tags for System.Net.Sockets

* Fixed XML documentation tags for Unix interop files

* Fixed XML documentation tags for System.Net.Http (Unix)

* Fixed XML documentation tags for PlatformDetection.Unix.cs according to review notes

* Fixed XML documentation tags for OSX interop

* Ignore CS1574 for System.Linq as there is no ConcatIterator{TSource}.ToList for uap.

* Fixed XML documentation tags for FreeBSD interop

* Fixed XML documentation tags for System.Net.WebSockets

* Fixed XMLm documentation tags for System.Transactions.Local

* Fixed XML documentation for System.Security.Principal.Windows

* Fixed XML documentation for System.Drawing.Common

* Fixed XML documentation for System.Net.NetworkInformation

* Fixed XML documentation tags for System.Private.XML (left CS1589)

* Fixed XML documentation tags for System.Security.Cryptography

* Fixed XML documentation for System.Runtime.WindowsRuntime

* Fixed XML documentation tags for System.Security.Cryptography.Xml

* Fixed XML documentation tags for System.Security.Cryptography.Primitives
CS3016 was added by mistake as part of #32934 so I removed it.

* Fixed XML documentation tags for System.Linq: added M to ToList method as it is not accessible by uap builds.

* Fixed XML documentation tags for System.IO.FileSystem

* Fixed XML documentation tags for System.Security.Cryptography.*

* Fixed XML documentation for System.IO.Ports

* Fixed XML documentation tags for System.Private.Reflection.Metadata.Ecma335

* Fixed XML documentation tags for System.Windows.Extensions

* Fixed XMl documentation tags for System.Console

* Fixed XML documentation for System.IO.FileSystem.DriveInfo

* Fixed XML documentation tags for System.Runtime.Extensions

* Fixed XML documentation tags for System.Drawing.Primitives.
Left CS1589 as there is no 'doc\KnownColor.uex' file.

* Fixed XMl documentation for System.Runtime.WindowsRuntime.
Had to add M prefix to 'Windows.Foundation.IAsyncAction' as Windows.Foundation is not accessible at compilation time.

* Fixed XML documentation tags for System.Private.Xml

* Fixed XML documentation tags for System.Private.Reflection.Metadata.Ecma335
Had to add a prefixes to: 'T:System.Collections.Immutable' and 'M:Enumerable.ToArray{TSource}' as they are not available for uap at compilation time.

* Fixed XML documentation tags for System.Net.WebSockets.WebSocketProtocol

* Fixed XML documentation tags for System.Net.WebSockets.WebSocketProtocol

* XML documentation tags fixes: changes after review.

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Do not use CorHost2::m_fStarted to gate execution (#20953)

* ExecuteInDefaultAppDomain w/o Start

Do not require Start() call before
calling ExecuteInDefaultAppDomain()

* Remove unused variable
* Move flags as needed to CorHost2

Merge pull request #21574 from BruceForstall/AddDotnetDiagnosticsInfo

Add dotnet info to PMI asm diffs output

Merge pull request #21572 from BruceForstall/DisableArm64Test

Disable System.Runtime.Tests for Linux arm64 corefx tests

Merge pull request #21532 from BruceForstall/FixPython3Issue

Fix Python 3 issues

Add tests for marshalling a System.Drawing.Color. (#21569)

Add dotnet info to PMI asm diffs output

This might help diagnose issues.

Disable System.Runtime.Tests for Linux arm64 corefx tests

Failure with System.Tests.ExceptionTests.ThrowStatementDoesNotResetExceptionStackLineOtherMethod
in JitStress=2.

Tracking: #21223

Disable GitHub_19361 for Linux arm32

It currently times out and fails on Debug.

Tracking: #20232

Switch to using resources in BenchmarksGames tests (#21554)

Embed the following test files as the resources in their corresponding test assemblies:

* regexdna-input25.txt regexdna-input25000.txt in BenchmarksGame/regex-redux

* knucleotide-input.txt knucleotide-input-big.txt in BenchmarksGame/k-nucleotide

* revcomp-input25.txt revcomp-input25000.txt in BenchmarksGame/reverse-complement

Test for #20657 SIMD byref bug

Delete dead code (#21562)

- Delete clix support from PAL. "clix" was launcher in the Rotor days - like "dotnet" today.
- Delete URL parsing from PALRT

Update dependency files (#21560)

* Update dependencies from https://github.com/dotnet/arcade build 802

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.18615.1
- Microsoft.DotNet.Helix.Sdk - 1.0.0-beta.18615.1

* Update dependencies from https://github.com/dotnet/arcade build 817

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.18616.5
- Microsoft.DotNet.Helix.Sdk - 1.0.0-beta.18616.5

Add specialized allocation helpers (#21561)

This is to support dotnet/corert#6688. They don't exist in CoreCLR (yet).

Merge pull request #21400 from BruceForstall/FixArm32FloatRange

Fix arm32 local variable references

Enable object stack allocation in R2R mode.

This change modified the importer to create GenTreeAllocObj node for
box and newobj instead of a helper call in R2R mode. ObjectAllocator phase
decides whether the object can be allocated on the stack or has to be created
on the heap via a helper call.

To trigger object stack allocation COMPlus_JitObjectStackAllocation has
to be set (it's not set by default).

Enable PinPlugAtGCTime event for standalone GC (#21543)

Update dependency files (#21508)

* Update dependencies from https://github.com/dotnet/arcade build 723

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.18611.14
- Microsoft.DotNet.Helix.Sdk - 1.0.0-beta.18611.14

* Update dependencies from https://github.com/dotnet/arcade build 751

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.18612.12
- Microsoft.DotNet.Helix.Sdk - 1.0.0-beta.18612.12

* Update dependencies from https://github.com/dotnet/arcade build 779

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.18613.20
- Microsoft.DotNet.Helix.Sdk - 1.0.0-beta.18613.20

* Update dependencies from https://github.com/dotnet/arcade build 791

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.18614.3
- Microsoft.DotNet.Helix.Sdk - 1.0.0-beta.18614.3

Update CoreClr, CoreSetup to preview-27215-01, preview-27214-02, respectively (#21547)

Fix more HWIntrinsic ifdefs (#21556)

Fix warning causing error in warning as error (#21557)

c:\github\coreclr\src\vm\excep.cpp(4489):
error C2220: warning treated as error - no 'object' file generated

[C:\GitHub\coreclr\bin\obj\Windows_NT.x64.Release\src\vm\wks\cee_wks.vcxproj]
c:\github\coreclr\src\vm\excep.cpp(4489):
warning C4800: 'BOOL': forcing value to bool 'true' or 'false'
(performance warning)
[C:\GitHub\coreclr\bin\obj\Windows_NT.x64.Release\src\vm\wks\cee_wks.vcxproj]

Fixing an assert in `gtGetStructHandleForSIMD` to validate the size of the simdType is less than or equal to the largest support TYP_SIMD (#21549)

Add Azure DevOps signing support (#21545)

Add signing support for Azure DevOps pipelines. This uses the arcade signing step, but with custom logic to find files to sign while we're not using arcade for the rest of the build.

Publish packages in Azure DevOps official build (#21536)

This enables coreclr participation in dependency flow.

Implement GC ref map parsing and display in R2RDump (#21509)

As part of my work on CPAOT implementation of GC ref map info
I have implemented decoder of the info so that it can be displayed
next to the import cell signatures.

This also uncovered one possible cause of R2RDump GcInfo-related
crashes that were observed by Andon and myself. It looks like Amy
in her initial implementation confused the various GC encodings
and used GC info to parse the import section auxiliary data which
actually contains the GC ref map info.

Thanks

Tomas

Merge pull request #21535 from CarolEidt/Fix21500

Update var life for multireg local

Simplifying the emitter handling of 4-byte encoded SSE instructions (#21528)

Update CoreClr, CoreFx, CoreSetup to preview-27213-02, preview.18613.4, preview-27213-01, respectively (master) (#21503)

* Update CoreClr, CoreFx, CoreSetup to preview-27213-02, preview.18613.4, preview-27213-01, respectively

* Add System.Threading.AccessControl to CoreFX test deps

Implement two pass algorithm for variant interface dispatch (#21355)

Fixes #20452.

Setting the m_willBlockOnGarbageCollectionEvent earlier and only when we will block to avoid the async break deadlock (#21319)

Setting the m_willBlockOnGarbageCollectionEvent earlier and only when we will block to avoid the async break deadlock

'mscorlib' link returns 404 -> 'System.Private.CoreLib' (#34086)

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Running and visualizing tests in AzureDevOps (Part 1) (#21502)

* Enable Azure Pipelines reporter so the test results can be seen at one place (i.e. "Tests" tab) - TestRunNamePrefix should encode all the needed information for a test run to be identifiable.

* Scenarios property should use comma (not semi-colon) to work correctly on non-Windows.

* Specify timeout to test jobs (the numbers are taken from netci.groovy) - we would adjust them as soon as we get this running in each PR.

* Specify timeout for Helix work items (the number is taken from buildpipeline) - same as above.

* Specify helixType and helixRepo properties to the job and use _HelixType and _HelixSource variables in MSBuild project instead of defining them manually.

*  Limit Pri0 Checked test jobs to running against pull requests. This also should be limited in the future to "default" build definition name - so when we have PR workload selection feature working we can distinguish between default PR validation and user triggered jobs.

* Remove "Prepare tests for helix" step - instead of archiving Core_Root and tests folders ourselves - directly specify them as correlation payload and work item payload directories and let the archiving be handled by Helix Sdk.

Update var life for multireg local

When a multi-reg var is defined by a call, but doesn't currently reside in a register,
we must still update liveness.

Fix #21500

Fixes issue #21485 and #21484 (#21526)

Issue #21485: fix EnumProcessModules hPseudoCurrentProcess bug.

Added handle reference.

Issue #21484: createdump segfaults with ASP.NET app

The problem is the ClrDataModule Request faulted on a dynamic module
getting the file layout flag.

Fixed the Request code not get the file layout and in the crash dump
code skip any dynamic modules.

JIT: avoid folding operations with relocatable immediates (#21511)

In general, don't fold operations on relocatable immediates. Only allow EQ/NE folding, since relocation should preserve identity but not bit values or relative comparisons.

Closes #21483.

Fix Python 3 issues

Fixes #21433

Minor cleanup of the interoputil.h API (#21476)

Suppress warning of obsolete API (#21529)

Merge pull request #21527 from CarolEidt/FixIfdef

Fix HWIntrinsics ifdef

Fix HWIntrinsics ifdef

Fix hardware intrinsc API XML comments (#21515)

* Fix AVX2 and SSE2 XML comments

* Fix ARM intrinsic XML comments

Fixing up configEnableISA and Compiler::compSetProcessor to be consistent with EEJitManager::SetCpuInfo in how ISA support checks are done (#21499)

Merge pull request #21304 from CarolEidt/NonObjSIMD

Don't require BLK nodes for SIMD

Suppress unused warnings for events raised by the VM (#21520)

Merge pull request #21512 from briansull/fix_750014

Fix for PMI assert:  'vnWx != NoVN'

JIT: optimize unbox/unbox.any when type is known (#21501)

Optimize away the unbox type test when the jit knows the type of object
being unboxed and can resolve the type comparison at jit time.

Closes #14473.

JIT: handle byref-typed immediates in encoding (#21477)

We may now retype some immediates (for instance RVA statics feeding into Spans)
as byrefs.  These need to be tracked as such during encoding so that the encoder
GC liveness tracking stays current.

Fixes the assert seen in #21466.

Merge branch 'master' into fix_750014

Fix for PMI assert:  'vnWx != NoVN'

Optimize number parsing (#21365)

Update dependencies from https://github.com/dotnet/arcade build 708 (#21487)

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.18610.4
- Microsoft.DotNet.Helix.Sdk - 1.0.0-beta.18610.4

Added support for debugging a sandboxed app on Mac (#21068)

* Fixed bug in StackString where the size is not initialized correctly to STACK_COUNT

* Added CharString and WCharString template classes and a generic CharStringFromLPCWSTR method

* Added support for debugging a sandboxed app on Mac

This change fixes the usage of IPC while debugging while running in a sandbox. When running in a sandbox, the temporary folder for each process will be different. Thus the pipes being created in TwoWayPipe right now would be created in different directories in the debugger process and the process being debugged.
This change configures the folder to be used based on the application group ID that the sandboxed app belongs to.
For the same reasons, the names sempahores being used to synchronize the debugger attach need to be prefixed with the application group ID. This change was abit more involved since the name of the semaphore is limited to 31 characters, so we had to encode the semaphore names differently to make them shorter.

Last, new APIs to the debugger shim were added to support this new feature. This change only handles the runtime side and the dbgshim. An additional change to vsdbg needs to be done to use the new APIs.

fixes #21066

* Fixed build breaks on non Mac Unix platforms

* Fixed usage of gApplicationGroupId in non apple environments

* Fixed bug in semaphore names

* Got rid of usage of StackString

* Made PAL_GetApplicationGroupId Apple specific

* Added comment about pragma pack

* Fixed comment

* Added exported symbols

* Duplicated applicationGroupId so it can be used from another thread during register complete callback

* Renamed BitNum2ByteNum to GetExtraEncodedAreaSize to make intent clearer

* Fixed nit comments

* Removed redundant changes in StackString

* Fixed windows build break

* Fixed compilation switch from __APPLE to __APPLE__

* Added missing exports

JIT: capture failing inline observation in more cases (#21471)

We were not capturing failing inline observations for failed inlines
that got reported by the runtime directly. Move the point of capture
up in the jitted code so it always happens regardless of reporting
obligations.

Moving the remaining Vector128/256 helper intrinsics to be implemented using other intrinsics (#21451)

* Updating Vector256.Create(V128, V128) to be implemented using other intrinsics

* Updating Vector128.Create(T, ...) and Vector256.Create(T, ...) to be implemented using other intrinsics

* Renaming CreateSoftware to SoftwareFallback and adding clarifying comments to the helper method code

* Fixing Vector128.Create(int, int, int, int) to properly consume the elements for the Sse2 codepath

Update CoreClr to preview-27211-02 (#21488)

Update CoreClr to preview-27211-01 (#21481)

Delete unnecessary struct initialization (#21482)

make GC/GCHandle tracing pay-for-play (#21479)

Fix for SetGCHandle event not getting emitted when running under Standalone GC (#21478)

Eliminate GenTreeRegVar and GT_REG_VAR and RegVar (#18317)

Issue #18201 / Hackathon

Adding support for the 2-byte VEX encoding to the emitter (#21453)

* Adding support for the 2-byte VEX encoding to the emitter

* Relocating and expanding the comment which explains the two-byte VEX encoding check

Merge pull request #21386 from briansull/issue_19925

Preserve the exception sets from child nodes when we create a new unique value number

Specify HelixTargetQueues in yaml (#21443)

* Parametrize test-job with helixQueues(Public|Internal) in eng/test-job.yml and specify them in eng/platform-matrix.yml

* Remove HelixTargetQueues property in tests/helixpublishwitharcade.proj

* Fix the error "unknown os: linux2" with Python script

Don't require BLK nodes for SIMD

Eliminate most cases where an OBJ or BLK node is required for SIMD
values. The exception is the case where a value produced by an intrinsic
(SIMD or HWIntrinsic) is used as an argument but the argument is of a
different SIMD type (e.g. a different baseType),

Fix warnings as errors

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Refactor internal System.AppDomain out of CoreLib (#21460)

Fixes #21028

Renamed VNCollectExc to VNUnionExcSet

Merge branch 'master' into issue_19925

Instead of using Win8+ api-sets use forwarder DLL to support downlevel versions of Windows. (#21469)

Update CoreClr to preview-27210-02 (#21467)

Add support for RVA fields to crossgen/R2R (#21463)

* Add support for RVA fields to crossgen/R2R

RVA fields are became more common with pre-inititialized ReadOnlySpan<byte>. Fix crossgen to deal with them for R2R.

* Fix tests, map new JIT helper for R2R

Update dependency files (#21448)

* Update dependencies from https://github.com/dotnet/arcade build 677

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.18607.6
- Microsoft.DotNet.Helix.Sdk - 1.0.0-beta.18607.6

* Update dependencies from https://github.com/dotnet/arcade build 679

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.18608.1
- Microsoft.DotNet.Helix.Sdk - 1.0.0-beta.18608.1

* Update dependencies from https://github.com/dotnet/arcade build 679

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.18608.1
- Microsoft.DotNet.Helix.Sdk - 1.0.0-beta.18608.1

Update CoreClr, PgoData to preview-27210-01, master-20181210-0037, respectively (#21452)

Fix for #21456 (Regressions in attribute allocations for non-generic attributes) (#21462)

* Fix for #21456 - restrict increased generic attribute allocations to only generic attributes

This is a trivial quick-fix for #21456 where regressions between 2.1 and 3.0 were discovered on most attibute pathways due to the allocation overhead in the generic-supporting pathways. The workaround is to simply not take that slow/expensive path for non-generics.

While I'd like to optimize `RuntimeModule.ResolveMethod` further, there's a public surface area in play there that makes the changes non-trivial. There, we'll have to choose overhead on the public path (which may still be a net win), or duplication in code for another path.

* Update comments

Delete vm/context.* (#21459)

* Delete vm/context.*

Leftover from remoting

Added Close On Exec Attribute for Mutex files (#21455) (#21458)

* Added Close On Exec Attribute for mutex files (#21455)

Fixed bug where "DCI_NUMA" was missing from debug channel names (#21454) (#21457)

In additional to adding "DCI_NUMA" to dbg_channel_names, a static assertion was added to verify that  dbg_channel_names length is valid. Also added some comments to help developpers not to forget to add corresponding entries in dbg_channel_names.

FIxes #21454

Update CoreClr, CoreFx, CoreSetup, PgoData to preview-27208-01, preview.18606.1, preview-27206-02, master-20181208-0038, respectively (#21374)

Fix regression in fat calls (#21446)

This seems to fix an AV in RyuJIT.

Leap Seconds Support (#21420)

* Leap Seconds Support

* Address Feedback

* More feedback addressing

Invalid setting csharp_new_line_within_query_expression_clauses in .editorconfig (#21449)

Same as in https://github.com/dotnet/corefx/issues/25991.

Roslyn doesn't support csharp_new_line_within_query_expression_clauses.
But Roslyn does support csharp_new_line_between_query_expression_clauses and it's missing from .editorconfig.

I also removed the comment "# Trim trailing whitespace" because that option is not set.