[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