On 05/02/2021 17.23, Peter Maydell wrote:
On Fri, 5 Feb 2021 at 16:21, Stefan Hajnoczi <stefa...@redhat.com> wrote:
Thanks, I update the patch in question.
It looks like the GitLab CI doesn't include a clang version that
produces this error because the pipeline passed for me:
https://gitlab.com/stefanha/qemu/-/pipelines/251524779
Is there something clang-specific you want to check in the CI? Maybe
clang 3.4, the oldest version supported according to ./configure?
Would probably be nice I guess. My ad-hoc builds use clang 6,
which is what tripped up here.
We should maybe discuss first whether we can bump the minimum version of
Clang that we would like to support. I once picked Clang 3.4 since that was
available in EPEL for RHEL7, but I think there were newer versions of Clang
available in RHEL7 via other repos later, so 3.4 is likely really just way
too old now...
According to https://developers.redhat.com/HW/ClangLLVM-RHEL-7 there was at
least Clang 7.0 available on RHEL7. Debian stable seems to have at least
7.0, too, according to repology.org. Ubuntu 18.04 seems to have version 6,
but later ones are available via updates? Anyway, I think we could at least
bump the minimum version to 6.0 nowadays...
Thomas
