🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/158623

• 📄 Preview Python docs built from this PR
• 📄 Preview C++ docs built from this PR
• ❓ Need help or want to give feedback on the CI? Visit the bot commands wiki or our office hours

Note: Links to docs will display an error until the docs builds have been completed.

❌ 9 New Failures, 1 Cancelled Job, 15 Pending

As of commit cfa4291 with merge base 0d40ff3 ():

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@pytorchbot rebase -b main

@pytorchbot started a rebase job onto refs/remotes/origin/main. Check the current status here

Successfully rebased llvm onto refs/remotes/origin/main, please pull locally before adding more changes (for example, via git checkout llvm && git pull --rebase)

I do kind of think it probably would be ok to drop OSS testing

@cyyever we can't find much usage of this indeed.
Let's drop the test altogether!

@albanD What files did you mean?

@albanD @ezyang Did you mean remove all of test/cpp/tensorexpr?

Yes. So you can remove the whole llvm9 requirement

@albanD @ezyang Internal builds still rely on the tests, so should we consider this PR?

wdym internal builds?

wdym internal builds?

#158928 has been reverted because the tensorexpr tests are used internally.

@cyyever I didn't had time to forward fix before it was reverted. From what I've seen, it is mostly build file update and utils being used from the deleted code.

+1 on what @albanD and @ezyang have said: let's just stop testing it in OSS altogether, and I can take a stab at removing the code altogether on Friday (it's probably not used internally at all, but build rules for it still exist

@cyyever I guess this needs clean rebase on top of trunk

@albanD Done