Closed surovic closed 1 year ago
Is it recorded somewhere what version of the compiler-rt that is included here? I think it would be good to add that information somewhere.
I'll make an issue to add a note in a new polytracker/src/compiler-rt/README.md
.
I've assumed that anything in
compiler-rt
is untouched so I haven't paid to much attention to those files. Is that correct?
Yes. That is correct.
The ports over the
DataflowSanitizer
pass and related parts ofcompiler-rt
from our modified fork of the LLVM repository. By doing this we are no longer dependent on that fork and can eventually move to more recent versions of LLVM. For this to happen, parts of polytracker's CMake build system had to be modified and the Dockerfile had to be rewritten. Users should also expect increased docker image build times as we now buildlibc++
, which was part of thepolytracker-llvm
docker dependency build. The sum duration ofpolytracker-llvm
andpolytracker
docker image build has however been drastically lowered. Future hope is to optimize the docker image build further.