search for: emitframemov

...n 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 there are probably other dependencies. In 2.7...

...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 there are probably other dependencies...

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

...ction; > + > +public: > + JITDwarfEmitter(JIT& jit); > + > + unsigned char* EmitExceptionTable(MachineFunction* MF, > + unsigned char* StartFunction, > + unsigned char* EndFunction); > + > + void EmitFrameMoves(intptr_t BaseLabelPtr, > + const std::vector<MachineMove> &Moves); > + > + unsigned char* EmitCommonEHFrame(const Function* Personality); > + > + unsigned char* EmitEHFrame(const Function* Personality, > + unsigned cha...

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:

...= JITMemoryManager::CreateDefaultMemManager(); > + } > + > + unsigned char* EmitExceptionTable(MachineFunction* MF, > + unsigned char* StartFunction, > + unsigned char* EndFunction); > + > + void EmitFrameMoves(intptr_t BaseLabelPtr, > + const std::vector<MachineMove> &Moves); > + > + unsigned char* EmitCommonEHFrame(const Function* Personality); > + > + unsigned char* EmitEHFrame(const Function* Personality, > + unsi...

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

...faultMemManager(); >> + } >> + >> + unsigned char* EmitExceptionTable(MachineFunction* MF, >> + unsigned char* StartFunction, >> + unsigned char* EndFunction); >> + >> + void EmitFrameMoves(intptr_t BaseLabelPtr, >> + const std::vector<MachineMove> &Moves); >> + >> + unsigned char* EmitCommonEHFrame(const Function* Personality); >> + >> + unsigned char* EmitEHFrame(const Function* Personality, >> +...