BPatch_binaryEdit::staticExecutableLoaded was poorly named. Changing
authorDaniel McNulty <mcnulty@cs.wisc.edu>
Wed, 28 Apr 2010 17:52:53 +0000 (12:52 -0500)
committerDaniel McNulty <mcnulty@cs.wisc.edu>
Wed, 28 Apr 2010 17:52:53 +0000 (12:52 -0500)
commit40550ec3a2572df9586b3ae060f0f8cf0cf357ab
tree5288ed8720c06984c726fce1b91dacfee6f7fdf5
parent79fe99124253cd14dbe26af4d08894faad0ab49d
BPatch_binaryEdit::staticExecutableLoaded was poorly named. Changing
its name to isStaticExecutable (per feedback from others in the group).
This also involves changes to the testsuite.

Fixed some bugs in the rewriter of static binaries:
- was using replaceFunctionCall to replace ctor and dtor handlers -- this
  appears to be having some problems on x86. Instead just instrumented the
  original ctor/dtor handler with a funcJump to the new handler. In retrospect,
  this is the better solution.

- Variables in relocatable files that are in COMMON were not being created
  correctly by Symtab. If a variable is in COMMON, there is no way to have an
  alias for this variable so new variables are created for every COMMON symbol
  that corresponds to a variable.
dyninstAPI/h/BPatch_binaryEdit.h
dyninstAPI/src/BPatch_binaryEdit.C
dyninstAPI/src/linux-x86.C
symtabAPI/src/Symtab.C
symtabAPI/src/emitElf-64.C
testsuite/src/dyninst/test1_14.C
testsuite/src/dyninst/test1_21.C
testsuite/src/dyninst/test1_22.C