search for: trampmem

...cast<IntrinsicInst>(Callee); > + if (!AdjustTramp || > + AdjustTramp->getIntrinsicID() != Intrinsic::adjust_trampoline) > + return 0; > + > + // The first argument passed to adjust_trampoline is the trampoline argument. > + // Get a hold of this. > + Value *TrampMem = AdjustTramp->getOperand(0); > + > + // The first thing to check is if there is a direct path from a > + // init.trampoline instruction to this adjust.trampoline instruction (both > + // operating on the same trampoline). Of course, this is useless if there are > + // writes...

Hi! Attached patches split init.trampoline into adjust.trampoline and init.trampoline, like in gcc. As mentioned in the previous mail, I've not made a documentation patch, since I'm not sure about what the documented semantics of llvm.adjust.trampoline should be. Thanks! -- Sanjoy Das http://playingwithpointers.com -------------- next part -------------- A non-text attachment was

...); This will crash if it isn't an intrinsic. Use dyn_cast not cast. > + if (AdjustTramp == NULL) > + return NULL; You forgot to check that this intrinsic is the adjust.trampoline intrinsic. Also, the LLVM style is to use 0 rather than NULL in this kind of situation. > + Value *TrampMem = AdjustTramp->getOperand(0); Maybe you should append a ->stripPointerCasts() to the line defining TrampMem, since it may have had to be bitcast to match the type of the adjust pointer argument. While the init.trampoline takes the same type as adjust.trampoline, it might have it's own b...

Hi! Attached set of patches splits llvm.init.trampoline into an "init" phase and an "adjust" phase, as discussed on the "Go on dragonegg" thread. Thanks! -- Sanjoy Das http://playingwithpointers.com -------------- next part -------------- A non-text attachment was scrubbed... Name: 0001-Split-intrinsics-and-DAG-nodes.patch Type: text/x-diff Size: 8808 bytes Desc: