<table border="1" cellspacing="0" cellpadding="8">
    <tr>
        <th>Issue</th>
        <td>
            <a href=https://github.com/llvm/llvm-project/issues/63252>63252</a>
        </td>
    </tr>

    <tr>
        <th>Summary</th>
        <td>
            clang can occasionally, flakily find `#include`d files that don't exist on disk when using `-fmodules-embed-all-files`
        </td>
    </tr>

    <tr>
      <th>Labels</th>
      <td>
            new issue
      </td>
    </tr>

    <tr>
      <th>Assignees</th>
      <td>
      </td>
    </tr>

    <tr>
      <th>Reporter</th>
      <td>
          zygoloid
      </td>
    </tr>
</table>

<pre>
    When files are embedded into an AST file, and then the corresponding `SLocEntry` gets lazily loaded from that AST file, we register the file from the AST file with the `FileManager` so that we can override its contents. But this causes flaky behavior if the file doesn't actually exist on disk: a `#include` that names that file in *just the right way* will work. Eg:

- The AST file contains an embedded copy of `foo/bar.h`, which doesn't exist on disk.
- For some reason, we load in an `SLocEntry` referring to that file. This can happen in very subtle and fragile ways, eg due to #63228.
- Loading that file registers it with `FileManager` and overrides its contents.
- A later `#include "bar.h"` will work, if we have `-iquote foo`, because we'll look for `foo/bar.h` and the `FileManager` will produce one.
- But a later `#include "bar.h"` will *not* work if we have `-iquote ./foo`, and similarly `#include "foo/bar.h"` will not work if we have `-iquote .`, because `./foo/bar.h` is *not* registered with `FileManager`. (If the file exists on disk, these file names work because we map them to the same `FileEntry`, whose contents are overridden.)

Perhaps we shouldn't associate the virtual file the a file name in `FileManager::SeenFileEntries` in this case, so that later queries for the same file name will still go to disk and fail? (I think we still need to create a virtual file so that we can override its contents and provide a corresponding `FileEntry`).
</pre>
<img width="1px" height="1px" alt="" src="http://email.email.llvm.org/o/eJyMVV-P4roP_TThxaJq06EDDzywM4v0k_YnXWlGus-mcdssacImKVz20185pfwRu1f7UlCT2sfnHNsYgm4t0VosvojF-wyH2Dm__nlunXFazXZOndd_d2Sh0YYCoCegfkdKkQJtowO0sPn4TMdCvgFaBZHvx46gdt5TODirtG1BVPnHN1d_tdGfRZVDSzGAwZ_anME45IiNdz3EDuNDzBOBp1aHSD6F5ffTVbrehJOOXXojqnyrDf0fLbbkOVVwY9QTQY0W3JG814pAxwC1s5FsDBl8GSLETgeocQgUoDG4P8OOOjxq50E3t-zKUbBCvkbAOg5ozBnoHx0iOAtKh70oN4AMRMhS29oMihhHAmGxpzD-TbG0BSE334cQU3yv2y7CCc9CbuCkjYGT8_sMvrai3Ij8XeSX5xw-78vnOlDbwIpcJard4QyuYSSNc0Jud-izLuF6g1On6-6ulIcKsinJ1nkIrmcNMDh7EYQFY-Ron3T11DC9toXobmVm8DlSa6HDw4Esf30kf4Yw7KKh5JzGY5ukxHPgRNSCGojjCFlWpZTLK6xvDpOrbjxOHgmg42iGZyNwkkn98Cj_FHgDBtlpD-KBkHKkTkqOc9WFUeqGGenwmKw31z8GFwmY75HnHSVDwYmEfDUGjHN7aJx_VmVqn18gTxkP3qmhJnCWrnjZtfjHmIXcWBeTtZzf_wZ6JuT2Bp8xBd1rg96cn1PcV3CXyLr43ykeuRFVPmW9o0OHO8CTvqR-o24GQi7_d9elydDh2pPyjY_C5XBswwTxJhD0eOBL_ehdgoD9VY3J4WPvuEBX86S5ePGVIpsJubrv1L_Id3gIHD90bjDqMjpCcLXGSCnTUXueJCM4foE3nGlGPJZbbkS5-SCyEzJNIVFmpxkW0vCcRt9okB8D8cXkvmt5tzRJuRD52TqmgHkbOxO1EeU2McwJ7D5Vk65aIsWXa09cDD6W8iezN6U4eHfkA3zeG4_sr7KZWpdqVa5wRuuiWlaLopCLctatscoV0WteFc3yZdcUBVVFgUS4Uqpc0Wqm1zKXZV4VRbGS-aLM6mWxXOVLSYsX-VKQEi859ahNZsyxz5xvZzqEgdZVKRdyZnBHJqRtKaWlE6RD9v3ifebX_M18N7RBvOSGvXeLEnU0tK4N2nbkoa4xaGd5d7BOvGx4EzbaqqfFoS7rNxGp3C_GNZx46Q7hQti86Z0aDIV5WgVzNGaeQogqnw3erLsYD4FNJLdCblsdu2GX1a4XcsuQLz_zg3ffqY5CblOhQchtIuLfAAAA___m9bhB">