On 19 October 2016 at 18:51, Tobias Droste <tdro...@gmx.de> wrote:
> Am Mittwoch, 19. Oktober 2016, 13:55:16 CEST schrieb Emil Velikov:
>> On 16 October 2016 at 01:20, Tobias Droste <tdro...@gmx.de> wrote:
>> > No need for an additional function here.
>> > Use the same style for LLVM checks as the other drivers
>> > (e.g. r300, llvmpipe) that don't need a load of other checks.
>> > Instead of open conding the LLVM version check, use the
>> > function used by other drivers.
>> >
>> > "enable_gallium_llvm" is checked by gallium_require_llvm().
>> >
>> > Signed-off-by: Tobias Droste <tdro...@gmx.de>
>> > ---
>> >
>> > configure.ac | 13 ++-----------
>> > 1 file changed, 2 insertions(+), 11 deletions(-)
>> >
>> > diff --git a/configure.ac b/configure.ac
>> > index 6a1e5c4..35034d3 100644
>> > --- a/configure.ac
>> > +++ b/configure.ac
>> > @@ -2427,16 +2427,6 @@ radeon_gallium_llvm_check() {
>> >
>> > radeon_llvm_check $*
>> >
>> > }
>> >
>> > -swr_llvm_check() {
>> > - gallium_require_llvm $1
>> > - if test ${LLVM_VERSION_INT} -lt 306; then
>> > - AC_MSG_ERROR([LLVM version 3.6 or later required when building
>> > $1]) - fi
>> > - if test "x$enable_gallium_llvm" != "xyes"; then
>> > - AC_MSG_ERROR([--enable-gallium-llvm is required when building
>> > $1])
>> > - fi
>> > -}
>> > -
>> >
>> > swr_require_cxx_feature_flags() {
>> >
>> > feature_name="$1"
>> > preprocessor_test="$2"
>> >
>> > @@ -2545,7 +2535,8 @@ if test -n "$with_gallium_drivers"; then
>> >
>> > fi
>> > ;;
>> >
>> > xswr)
>> >
>> > - swr_llvm_check "swr"
>> > + llvm_check_version_for "3" "6" "0" "swr"
>> > + gallium_require_llvm "swr"
>>
>> Nit: Might be worth flipping the two ?
>> I can sort that one before pushing.
>
> I order it that way so you get the error message with the swr LLVM version and
> not the gallium LLVM version (which is usually lower).
> But I can change it if you want.
>
> Always forgetting the list... sorry for sending it twice.
>
Don't bother with it.
Thanks
Emil
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
