Track a missing place to reset alignment flag from BasicBlock

[kunalspathak]

There was a place when we reverse a conditional jump, we were not removing the alignment flag.

Environment:

set COMPlus_TieredCompilation=0
set COMPlus_JitAggressiveInlining=1

Assert failure(PID 10876 [0x00002a7c], Thread: 32880 [0x8070]): Assertion failed 'loopsToProcess == 0' in 'TestClass:Method60(byref,byref,byref,byref,S1_D1_F1,byref,byref,S2,byref,byref,byref,byref):S1:this' during 'Place 'align' instructions' (IL size 528)
    File: D:\git\runtime\src\coreclr\jit\compiler.cpp Line: 5297
    Image: D:\git\runtime\artifacts\tests\coreclr\windows.x64.Checked\tests\Core_Root\CoreRun.exe

There are some occurrences of this issue that Antigen caught over the weekend - https://dev.azure.com/dnceng/public/_build/results?buildId=1488159&view=ms.vss-build-web.run-extensions-tab

[ghost]

Tagging subscribers to this area: @JulieLeeMSFT
See info in area-owners.md if you want to be subscribed.

Issue Details

---

There was a place when we reverse a conditional jump, we were not removing the alignment flag.

Environment:

set COMPlus_TieredCompilation=0
set COMPlus_JitAggressiveInlining=1

Assert failure(PID 10876 [0x00002a7c], Thread: 32880 [0x8070]): Assertion failed 'loopsToProcess == 0' in 'TestClass:Method60(byref,byref,byref,byref,S1_D1_F1,byref,byref,S2,byref,byref,byref,byref):S1:this' during 'Place 'align' instructions' (IL size 528)
    File: D:\git\runtime\src\coreclr\jit\compiler.cpp Line: 5297
    Image: D:\git\runtime\artifacts\tests\coreclr\windows.x64.Checked\tests\Core_Root\CoreRun.exe

Author:	kunalspathak
Assignees:	-
Labels:	area-CodeGen-coreclr
Milestone:	-

[kunalspathak]

@dotnet/jit-contrib @BruceForstall

[BruceForstall]

LGTM.

btw, the JITDUMP in BasicBlock::unmarkLoopAlign() looks like it needs a trailing \n