This is an archive of the discontinued LLVM Phabricator instance.

[InjectTLIMappings] Preserve GLobalsAAWrapperPass.
AbandonedPublic

Authored by fhahn on Jun 19 2020, 7:59 AM.

Details

Summary

InjectTLIMappings runs before LoopVectorize but is not marked as
preserving GlobalsAA. This means that GlobalsAA is not available in
LoopVectorize, causing some loops to not be vectorized due to failing to
deduce no-alias for some accesses.

This fixes a few 5-10% regressions on SPEC2000/SPEC2006 with LTO & PGO.

Diff Detail

Event Timeline

fhahn created this revision.Jun 19 2020, 7:59 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 19 2020, 7:59 AM

Hi @fhahn ,

it seems there is some overlap with this patch? https://reviews.llvm.org/D82063

Francesco

fhahn abandoned this revision.Jun 19 2020, 8:43 AM

it seems there is some overlap with this patch? https://reviews.llvm.org/D82063

Oh right, that's great! What a coincidence! I'll abandon this one then.