I have cleaned up the code somewhat that Olivier wrote and split up the patch into three pieces. This first is to make the MCJIT not have to initialize all asm printers, but only the native one. - Jan -------------- next part -------------- A non-text attachment was scrubbed... Name: 0018_nativeasmprinterinit.patch Type: text/x-diff Size: 12993 bytes Desc: not available URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20100728/c7d7618f/attachment.patch>
Hi Jan, Applied with edits in r109996, thanks! I wasn't happy about the change to add "Target" everywhere -- I just added a local hack in TargetSelect.h to workaround this. We should really change the definition of LLVM_NATIVE_TARGET, but I am not in a configure hacking mood. - Daniel On Wed, Jul 28, 2010 at 10:39 AM, Jan Sjodin <jan_sjodin at yahoo.com> wrote:> I have cleaned up the code somewhat that Olivier wrote and split up the patch into three pieces. This first is to make the MCJIT not have to initialize all asm printers, but only the native one. > > - Jan > > _______________________________________________ > LLVM Developers mailing list > LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu > http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev > >
Thanks! I agree, it would have been nice to have only the target name in LLVM_NATIVE_TARGET, but I did not know how to do that and keeping the names somewhat consistent seemed like the next best option. - Jan ----- Original Message ----> From: Daniel Dunbar <daniel at zuster.org> > To: Jan Sjodin <jan_sjodin at yahoo.com> > Cc: LLVM Developers Mailing List <llvmdev at cs.uiuc.edu> > Sent: Sun, August 1, 2010 7:36:36 PM > Subject: Re: [LLVMdev] MC-JIT Patches 1/3 > > Hi Jan, > > Applied with edits in r109996, thanks! > > I wasn't happy about the change to add "Target" everywhere -- I just > added a local hack in TargetSelect.h to workaround this. We should > really change the definition of LLVM_NATIVE_TARGET, but I am not in a > configure hacking mood. > > - Daniel >