Re: [DynInst_API:] getLoopHead interface in BPatch_basicBlockLoop class

2015-07-28 Thread Jim Galarowicz
Hi all, I've downloaded the git source version of dyninst and built it on NASA's pfe machine and my laptop in an attempt to run it on Intel MIC binaries. The 8.2.1 asserts in switches related to the architecture type. However, I ran into a compile error with our loop code: [ 16%]

Re: [DynInst_API:] getLoopHead interface in BPatch_basicBlockLoop class

2015-07-28 Thread Xiaozhu Meng
It is more complicated than that. For a natural loop, it only has one entry, which is also called the head because the entry dominates all blocks of the loop. However, for an irreducible loop, it can have multiple entries and it is possible that none of the entries dominates the other blocks in

Re: [DynInst_API:] getLoopHead interface in BPatch_basicBlockLoop class

2015-07-28 Thread Barton Miller
Xiaozhu, Do we have an updated manual entry for the new getLoopEntries? --bart On 7/28/2015 10:37 AM, Xiaozhu Meng wrote: Hi Jim, The old getLoopHead() method has been replaced by the following new interface to appropriately represent irreducible loops: int

Re: [DynInst_API:] getLoopHead interface in BPatch_basicBlockLoop class

2015-07-28 Thread Xiaozhu Meng
Hi Jim, The old getLoopHead() method has been replaced by the following new interface to appropriately represent irreducible loops: int BPatch_basicBlockLoop::getLoopEntries(BPatch_VectorBPatch_basicBlock* e); The new interface returns the number of entry blocks of this loop and the entries

Re: [DynInst_API:] PATCH: have SymtabAPI properly use ELF contents to set architecture

2015-07-28 Thread Jim Galarowicz
Hi Bill, I've built dyninst from the latest git bits for dyninst and then changed our source to get around the loop head API issue. When I run using the new version of dyninst, I'm still getting the architecture assert shown below. Is there anything I can try to get around this issue? Jim G

Re: [DynInst_API:] PATCH: have SymtabAPI properly use ELF contents to set architecture

2015-07-28 Thread Bill Williams
On 07/28/2015 01:47 PM, Jim Galarowicz wrote: Hi Bill, all, I cancelled my original post - too large. Resending a shortened version.. I sent mail to a NASA representative about the elf.h EM_ architecture enumerated value. The elf.h files that I've been able to locate only go up to the

[DynInst_API:] commit: applying two external patches

2015-07-28 Thread Bill Williams
From Rashawn Knapp at Intel: Enable building Dyninst with ICC From John Mellor-Crummey at Rice: Fix a fencepost error in SymtabAPI's DWARF parsing --bw Bill Williams Paradyn Project b...@cs.wisc.edu ___ Dyninst-api mailing list