[PATCH] D29654: [OpenMP] Integrate OpenMP target region cubin into host binary

[Gheorghe-Teodor Bercea via Phabricator via cfe-commits]

gtbercea added a comment.

In https://reviews.llvm.org/D29654#835507, @arphaman wrote:

> In https://reviews.llvm.org/D29654#835501, @gtbercea wrote:
>
> > Is that the last access to CachedResults before the error?
>
>
> Is the assertion the last access? Yes.
>
> There must be a discrepancy between
>
> `UI.DependentBoundArch` in the loop above and `BoundArch` that's used to compute `TargetTC`, otherwise `GetTriplePlusArchString` would return the key that matches the `0x0000000111c017d0` pointer, i.e. without the additional x86_64.


Maybe I misunderstood the output you pasted but it looks like ActionTC contains the same triple+arch that you can find in entry [1] in the map.


https://reviews.llvm.org/D29654