[clang] [clang][modules] HeaderSearch::MarkFileModuleHeader creates extra HeaderFileInfo, breaks PCM reuse (PR #89005)

[Sam McCall via cfe-commits]

sam-mccall wrote:

Hmm, I locally reverted https://github.com/llvm/llvm-project/pull/87849 and still see the same issue.
I'll patch #89428 instead, but I don't see how it could do better - it's just putting the old and new behavior of #87849 behind a flag, right?

In any case I'll try it, and then work on constructing a reproducer (so far I've been mostly treating this as a black box)

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