[llvm] r232139 - Fix build break in this code. Nothing uses this header, but the modules

Lang Hames lhames at gmail.com
Fri Mar 13 18:40:02 PDT 2015


Yep. As I mentioned in the other thread, I'll try to hook this up to LLI
soon.

Cheers,
Lang.

On Fri, Mar 13, 2015 at 3:02 PM, David Blaikie <dblaikie at gmail.com> wrote:

>
>
> On Thu, Mar 12, 2015 at 8:56 PM, Richard Smith <richard-llvm at metafoo.co.uk
> > wrote:
>
>> Author: rsmith
>> Date: Thu Mar 12 22:56:27 2015
>> New Revision: 232139
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=232139&view=rev
>> Log:
>> Fix build break in this code. Nothing uses this header, but the modules
>> buildbot builds it anyway and was angry because of this.
>>
>
> Heh, excellent.
>
> Lang - unit tests? (or how did this not have a problem in your
> Kaleidoscope tutorials? oh, right, they're not built by default and this
> was a recent API change that broke it)
>
>
>>
>> Modified:
>>     llvm/trunk/include/llvm/ExecutionEngine/Orc/CompileOnDemandLayer.h
>>
>> Modified:
>> llvm/trunk/include/llvm/ExecutionEngine/Orc/CompileOnDemandLayer.h
>> URL:
>> http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/ExecutionEngine/Orc/CompileOnDemandLayer.h?rev=232139&r1=232138&r2=232139&view=diff
>>
>> ==============================================================================
>> --- llvm/trunk/include/llvm/ExecutionEngine/Orc/CompileOnDemandLayer.h
>> (original)
>> +++ llvm/trunk/include/llvm/ExecutionEngine/Orc/CompileOnDemandLayer.h
>> Thu Mar 12 22:56:27 2015
>> @@ -274,7 +274,7 @@ private:
>>        // Set the compile actions for this module:
>>        for (auto &KVPair : NewStubInfos) {
>>          std::string BodyName = Mangle(KVPair->first + BodySuffix,
>> -                                      *M.getDataLayout());
>> +                                      M.getDataLayout());
>>          auto &CCInfo = KVPair->second;
>>          CCInfo.setCompileAction(
>>            [=](){
>> @@ -291,7 +291,7 @@ private:
>>
>>      for (auto &KVPair : StubInfos) {
>>        std::string AddrName = Mangle(KVPair.first + AddrSuffix,
>> -                                    *M.getDataLayout());
>> +                                    M.getDataLayout());
>>        auto &CCInfo = KVPair.second;
>>        CCInfo.setUpdateAction(
>>          CompileCallbackMgr.getLocalFPUpdater(StubsH, AddrName));
>>
>>
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150314/ba5ee8d4/attachment.html>


More information about the llvm-commits mailing list