add tt to mem calls - Githubissues

EnzymeAD / rust

A rust fork to work towards Enzyme integration

https://www.rust-lang.org

Other

52 stars 7 forks source link

add tt to mem calls #82

Closed ZuseZ4 closed 3 months ago

ZuseZ4 commented 3 months ago

wip, but didn't break anything for me. Let's ask CI

jedbrown commented 3 months ago

Does anyone understand why the cache is present, but not found?

Run actions/cache@v4
  with:
    path: build/build/x86_64-unknown-linux-gnu/llvm
    key: openstack22-llvm-606bc11367b475542bd6228163424ca43b4dbdbc
    enableCrossOsArchive: false
    fail-on-cache-miss: false
    lookup-only: false
    save-always: false
  env:
    CARGO_INCREMENTAL: 0
    CARGO_TERM_COLOR: always
Cache not found for input keys: openstack22-llvm-606bc11367b475542bd6228163424ca43b4dbdbc

jedbrown commented 3 months ago

Oh, caches have to be made on the same branch or the default branch. So this issue will go away as soon as we get a passing pipeline on master.

The cache action first searches for cache hits for key and the cache version in the branch containing the workflow run. If there is no hit, it searches for restore-keys and the version. If there are still no hits in the current branch, the cache action retries same steps on the default branch. https://docs.github.com/en/actions/using-workflows/caching-dependencies-to-speed-up-workflows#matching-a-cache-key

ZuseZ4 commented 3 months ago

@jedbrown Can you test this a bit? It should fix your thing and now also support slices.

jedbrown commented 3 months ago

Thanks, this is clean for me on two machines. I'll merge to master so we can seed that cache for future PRs.