[Libreoffice] [UNPUSHED][PATCH] Fix building bridge with -mavx

Norbert Thiebaud nthiebaud at gmail.com
Wed Aug 3 08:43:36 PDT 2011


2011/8/3 Tomáš Chvátal <tchvatal at novell.com>:
> Dne 3.8.2011 13:27, Caolán McNamara napsal(a):
>>
>> On Wed, 2011-08-03 at 12:11 +0100, Caolán McNamara wrote:
>>>
>>> On Wed, 2011-08-03 at 12:47 +0200, Tomáš Chvátal wrote:
>>>>
>>>> Attaching the updated patch that adds it only where it should really be
>>>> needed.
>>>
>>> seems reasonable, pushed.
>>
>> *bzzt*, x86_64 gcc 4.3.4 doesn't support it, so breaks the build on
>> those buildbots.
>>
>> I guess need to instead add a configure check for it, stuff it into an
>> env variable if supported, and add that env variable to the makefile.mks
>> instead.
>>
>> C.
>>
> Darn,
> Ok first part is the configure.in check, I think the attachment should do
> the trick.
>
> But I am not entirely sure how to do the check in plain GNUMake.
> I wrote it with ifdef USE_AVX and I hope this is the way it should work :)

I haven't read the patch, but you should use HAVE_GCC_AVX to stay
similar to existing check like that.

Norbert


More information about the LibreOffice mailing list