Tres

As of, https://github.com/openxla/xla/commit/ccceb5fefba9d946512c7a2041bbcc78d252364a, this is resolved.

Hi Nicholas, Can you comment on how this is used or built in the JAX side? I ask because the current build of Tensorflow with this change on mac fails....

Sorry for the delay on this. It took me a while to run some Tensorflow tests. Can you see if the following in `toolchains.patch` would work for your use case?...

Unfortunately, it was needed for building for iOS, but I will confirm I didn't make another mistake. Out of curiosity, have you tried making the file empty and seeing if...

I will follow up on this soon and add appropriate people. Tensorflow folks (who are still the largest contributors to XLA infrastructure) have already been working on upgrading to a...

It looks like I was mistaken that Tensorflow still needs this either unless I did something wrong. If you want to update your PR to remove this completely, I can...

You can fully remove it as long as you remove the reference to the patch file like here: https://github.com/openxla/xla/blob/main/third_party/llvm/workspace.bzl#L23

So unfortunately, do to the way we mirror code between Google and this github repository, i need to create a separate commit to do this. I will try to accredit...

https://github.com/openxla/xla/commit/a3e20fbb13887320f756c27b4f1ddae12e3e630c landed this with a bunch of other formatting changes for some reason