[llvm] [MemProf] Add option to match summary and definition strictly (PR #83665)

[Teresa Johnson via llvm-commits]

teresajohnson wrote:

> FYI I tried an internal target and I'm getting a failure in applyImport with this change. Let me see if I can identify the reason and will get back to you.

The error I am getting is the assert:
llvm/lib/Transforms/IPO/MemProfContextDisambiguation.cpp:3383 in bool llvm::MemProfContextDisambiguation::applyImport(Module &): SrcModuleMD && "enable-import-metadata is needed to emit thinlto_src_module"

I am passing -enable-memprof-context-disambiguation, however, so I would have thought your change to FunctionImport.cpp should force enable the addition of this metadata. When I explicitly add -enable-import-metadata the assert goes away, so that change isn't having the intended effect. I noticed your test still passes both of these flags, maybe try removing -enable-import-metadata to see if you can reproduce.

https://github.com/llvm/llvm-project/pull/83665