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

[Alex Lorenz via Phabricator via cfe-commits]

arphaman added a comment.

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.


https://reviews.llvm.org/D29654