[PATCH] D106781: [Zorg] Use ccache instead incremental build for openmp-offload-cuda-runtime.

[Shilei Tian via Phabricator via llvm-commits]

tianshilei1992 added a comment.

In D106781#2905956 <https://reviews.llvm.org/D106781#2905956>, @Meinersbur wrote:

> In D106781#2905360 <https://reviews.llvm.org/D106781#2905360>, @tianshilei1992 wrote:
>
>> Speaking of this, is it possible to integrate the result into Phabricator?
>
> The pre-merge buildkite already uses clean builds with ccache (or sccache). For small changes the incremental build typically is still by some factors faster than the ccache builds which is why I would prefer to keep incremental builds for `openmp-offload-cuda-project`. This also allows us to better identify problems that occur only in one of the builds (such as incorrect builds dependences) and still get frequent rebuilds.
>
> This is a general problem buildbots using LLVM_ENABLE_RUNTIMES builds <https://reviews.llvm.org/rZORG2ed0c6ae25a4da897f23770f9a4976c3e19fff10> and might be fixed some day by e.g. clobbering the runtimes build dir after clang had to be recompiled.

I mean, not about this patch. To integrate the execution result on GPUs into Phabricator. Currently we don't have test run on GPUs I think.


Repository:
  rZORG LLVM Github Zorg

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

https://reviews.llvm.org/D106781