[test][NewPM] Make dead-uses.ll work under NPM
authorArthur Eubanks <aeubanks@google.com>
Tue, 22 Sep 2020 22:18:55 +0000 (15:18 -0700)
committerArthur Eubanks <aeubanks@google.com>
Tue, 6 Oct 2020 14:33:02 +0000 (07:33 -0700)
commit61d4b342d15a82d5e841d9e72a52d77eaa5b5097
tree63386e531d3ad424fd39683103efc6d693245434
parent4a7a988442dfa40309b34746218c07ebf758b378
[test][NewPM] Make dead-uses.ll work under NPM

This one is weird...

globals-aa needs to be already computed at licm, or else a function pass
can't run a module analysis and won't have access to globals-aa.
But the globals-aa result is impacted by instcombine in a way that
affects what the test is expecting. If globals-aa is computed before
instcombine, it is cached and globals-aa used in licm won't contain the
necessary info provided by instcombine.
Another catch is that if we don't invalidate AAManager, it will use the
cached AAManager that instcombine requested, which may not contain
globals-aa. So we have to invalidate<aa> so that licm can recompute
an AAManager with the globals-aa created by the require<globals-aa>.

This is essentially the problem described in https://reviews.llvm.org/D84259.

Reviewed By: asbirlea

Differential Revision: https://reviews.llvm.org/D88118
llvm/test/Analysis/GlobalsModRef/dead-uses.ll