Formed in 2009, the Archive Team (not to be confused with the archive.org Archive-It Team) is a rogue archivist collective dedicated to saving copies of rapidly dying or deleted websites for the sake of history and digital heritage. The group is 100% composed of volunteers and interested parties, and has expanded into a large amount of related projects for saving online and digital history.
JIT: Align newly recognized loops #96756
Conversation
Also remove quirk when checking for inner-most loops.
ghost
assigned 
dotnet-issue-labeler
bot
added
the
area-CodeGen-coreclr
|
Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch Issue DetailsAlso remove quirk when checking for inner-most loops.
|
| // TODO-Quirk: Remove. When removing we will likely need to add some | ||
| // form of "lexicality" heuristic here: only align loops whose blocks | ||
| // are fairly tightly packed together physically. |
There was a problem hiding this comment.
I decided not to look at this here; in particular because we have a lot of work in flight that is severely changing the flow graph, so it's hard to get a good feeling for how this heuristic should eventually look.
|
cc @dotnet/jit-contrib PTAL @BruceForstall Diffs from a run with alignment enabled. Aggregated stats for our win-x64 SPMI collections: -Alignment candidates: 35898
+Alignment candidates: 38075
-Aligned: 13523
+Aligned: 141936.1% more candidates, 5.0% more aligned loops |
|
superpmi-replay failures are the timeouts we see every once in a while. The linker tests had a timeout in the macOS job. Other failures are known. |
Also remove quirk when checking for inner-most loops.
Also remove quirk when checking for inner-most loops.