[Mesa-stable] [Mesa-dev] [PATCH] clover: Fix build against clang SVN >= r273191

Michel Dänzer michel at daenzer.net
Wed Jun 22 01:18:21 UTC 2016


On 21.06.2016 22:36, Vedran Miletić wrote:
> On 06/21/2016 03:13 AM, Michel Dänzer wrote:
>> On 21.06.2016 08:17, Vedran Miletić wrote:
>>> setLangDefaults() now requires PreprocessorOptions as an argument.
>>> ---
>>>  src/gallium/state_trackers/clover/llvm/invocation.cpp | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
>>> b/src/gallium/state_trackers/clover/llvm/invocation.cpp
>>> index e2cadda..57eaaaa 100644
>>> --- a/src/gallium/state_trackers/clover/llvm/invocation.cpp
>>> +++ b/src/gallium/state_trackers/clover/llvm/invocation.cpp
>>> @@ -207,7 +207,7 @@ namespace {
>>>        c.getDiagnosticOpts().ShowCarets = false;
>>>        c.getInvocation().setLangDefaults(c.getLangOpts(),
>>> clang::IK_OpenCL,
>>>  #if HAVE_LLVM >= 0x0309
>>> -                                        llvm::Triple(triple),
>>> +                                        llvm::Triple(triple),
>>> c.getPreprocessorOpts(),
>>>  #endif
>>>                                         
>>> clang::LangStandard::lang_opencl11);
>>>        c.createDiagnostics(
>>>
>>
>> Pushed, thanks!
>>
>>
>> P.S. I recommend adding your Signed-off-by for patches you want to be
>> applied.
>>
> 
> Hi Michel and others,
> 
> this should probably go into 12.0 branch as well, so we at least make
> sure to support Clang/LLVM 3.9 git as of Mesa 12.0 release date.

No. We've been over this a couple of times before. :) Because the LLVM
C++ APIs can change in any way at any point of a development cycle, we
cannot support unreleased versions of LLVM in released versions of Mesa.


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer


More information about the mesa-stable mailing list