search for: tidylandingpad

...ied, but instead solely exists for identification purposes for those who know what they are doing. :-) The patch is offset from HEAD. The patch is a hack which removes use of the MCSymbol::isDefined(...) method, as its use happens to break in the exception JIT context; both in MachineModuleInfo.:: TidyLandingPads(...) and in JITDwarfException:: EmitFrameMoves(...). In this context these symbols are not defined but rather are temporaries. Beyond this I don't understand what the correct replacement is, or whether or not the use of MCSymbol::isDefined(...) can be dropped from TidyLandingPads(...) since t...

...ely exists for identification purposes for those who know what they are doing. :-) > The patch is offset from HEAD. > > The patch is a hack which removes use of the MCSymbol::isDefined(...) method, as its use happens to break > in the exception JIT context; both in MachineModuleInfo.:: TidyLandingPads(...) and in JITDwarfException:: > EmitFrameMoves(...). In this context these symbols are not defined but rather are temporaries. Beyond this > I don't understand what the correct replacement is, or whether or not the use of MCSymbol::isDefined(...) can be > dropped from TidyLandingPa...

Forgot to add platform with issue: OS X 10.6.2. I have not tested elsewhere. Garrison On Mar 18, 2010, at 14:20, Garrison Venn wrote: > Hi Chris, > > The MCSymbol r98459 patch of llvm seems to have broken the ExceptionDemo example. As the example is dying in the > associated personality's first unwind search phase, which happens to have no language specific context, and is

Hi Chris, The MCSymbol r98459 patch of llvm seems to have broken the ExceptionDemo example. As the example is dying in the associated personality's first unwind search phase, which happens to have no language specific context, and is returning a _URC_CONTINUE_UNWIND, I believe the issue is generic and not specific to the example. However I'm not sure why then this wasn't seen in one

Thx again Evan for the review. Here's a new patch for the JIT in itself. The major changes are: 1) A JITMemoryManager now has a flag saying "I require to know the size of what you want to emit" 2) DwarfJITEmitter is augmented with GetSize* functions 3) JITEmitter::startFunction checks if the JITMemoryManager requires to know the size. If so, it computes it and gives it through the

...ter::EmitExceptionTable > (MachineFunction* MF, > + unsigned char* > StartFunction, > + unsigned char* > EndFunction) { > + // Map all labels and get rid of any dead landing pads. > + MMI->TidyLandingPads(); > + > + const std::vector<GlobalVariable *> &TypeInfos = MMI- > >getTypeInfos(); > + const std::vector<unsigned> &FilterIds = MMI->getFilterIds(); > + const std::vector<LandingPadInfo> &PadInfos = MMI->getLandingPads > (); > + if...

...;> (MachineFunction* MF, >> + unsigned char* >> StartFunction, >> + unsigned char* >> EndFunction) { >> + // Map all labels and get rid of any dead landing pads. >> + MMI->TidyLandingPads(); >> + >> + const std::vector<GlobalVariable *> &TypeInfos = MMI- >> >>> getTypeInfos(); >>> >> + const std::vector<unsigned> &FilterIds = MMI->getFilterIds(); >> + const std::vector<LandingPadInfo> &...

Hi everyone, Here's a patch that enables exception handling when jitting. I've copy/pasted _many_code from lib/Codegen/DwarfWriter.cpp, so we may need to factorize it, but the functionality is there and I'm very happy with it :) lli should now be able to execute the output from llvm-gcc when using exceptions (the UnwindInst instruction is not involved in this patch). Just add the

...ter::EmitExceptionTable(MachineFunction* > MF, > + unsigned char* > StartFunction, > + unsigned char* > EndFunction) { > + // Map all labels and get rid of any dead landing pads. > + MMI->TidyLandingPads(); > + > + const std::vector<GlobalVariable *> &TypeInfos = MMI- > >getTypeInfos(); > + const std::vector<unsigned> &FilterIds = MMI->getFilterIds(); > + const std::vector<LandingPadInfo> &PadInfos = MMI- > >getLandingPads(); > + if...

Dear all, Here's a new patch with Evan's comments (thx Evan!) and some cleanups. Now the (duplicated) exception handling code is in a new file: lib/ExecutionEngine/JIT/JITDwarfEmitter. This patch should work on linux/x86 and linux/ppc (tested). Nicolas -------------- next part -------------- An embedded and charset-unspecified text was scrubbed... Name: jit-exceptions.patch URL: