[all-commits] [llvm/llvm-project] f0e10c: [HIP] Relax checks in hip-include-path.hip test ca...

Björn Pettersson via All-commits all-commits at lists.llvm.org
Mon May 31 13:41:02 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f0e10cc91bc40fb5c0698846f0a324d4658f54c7
      https://github.com/llvm/llvm-project/commit/f0e10cc91bc40fb5c0698846f0a324d4658f54c7
  Author: Bjorn Pettersson <bjorn.a.pettersson at ericsson.com>
  Date:   2021-05-31 (Mon, 31 May 2021)

  Changed paths:
    M clang/test/Driver/hip-include-path.hip

  Log Message:
  -----------
  [HIP] Relax checks in hip-include-path.hip test case again

The tightened checks from commit 722c39fef5ab6 did not work
fully for buildbots using symlinks in repo paths. This patch
is not fully reverting 722c39fef5ab6, as we still match that
there is a "/lib" somewhere in the path before "/clang/".

So this is once again a bit fragile in case someone would put
their repo in a base directory, for example, named
 "/scratch/lib/foo/clang/llvm-project/". But it is atleast a
bit better than the original checks (avoiding the problem that
commit 722c39fef5ab6 was solving).




More information about the All-commits mailing list