[PATCH] D92831: Update references to 'master' branch.

Hafiz Abid Qadeer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 10 04:20:28 PST 2020


abidh added inline comments.


================
Comment at: llvm/docs/CompileCudaWithLLVM.rst:147
 In clang, ``math.h`` and ``cmath`` are available and `pass
-<https://github.com/llvm/llvm-test-suite/blob/master/External/CUDA/math_h.cu>`_
+<https://github.com/llvm/llvm-test-suite/blob/main/External/CUDA/math_h.cu>`_
 `tests
----------------
kristof.beyls wrote:
> For the llvm-test-suite project (rather than the llvm-project project), there is no main branch yet.
> So if we'd commit this as is, that would break the link.
> 
> @sqlbyme did you have a timeline for when you'd expect to also add a main branch in llvm-test-suite.
> If that is expected to happen soon, maybe it's simplest to just wait for that to happen and then commit this patch as a whole?
Thanks for catching this. It was an oversight. I will remove it from the patch as there are other locations which refers https://github.com/llvm/llvm-test-suite which I had not touched. We can update them when they have a main branch of their own.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D92831/new/

https://reviews.llvm.org/D92831



More information about the llvm-commits mailing list