[DynInst_API:] [dyninst/dyninst] ccebb6: More annotation code to suppress false positives o...

2018-07-23 Thread Xiaozhu Meng 
  Branch: refs/heads/new-parallel-parsing
  Home:   https://github.com/dyninst/dyninst
  Commit: ccebb687f1f2a893755ea1bb2970d1eeb9cb32a4
  
https://github.com/dyninst/dyninst/commit/ccebb687f1f2a893755ea1bb2970d1eeb9cb32a4
  Author: Xiaozhu Meng 
  Date:   2018-07-23 (Mon, 23 Jul 2018)

  Changed paths:
M parseAPI/h/CFG.h
M parseAPI/h/LockFreeQueue.h
M parseAPI/src/CFGFactory.C
M parseAPI/src/Function.C
M parseAPI/src/ParseData.C

  Log Message:
  ---
  More annotation code to suppress false positives of data race reports
from cilkscreen

1. add fake locking for std::atomic
2. remove access history for objects initialized by one thread and then
shared


  Commit: 90304602fcc4b2f61f8834861b71832d8f2c3135
  
https://github.com/dyninst/dyninst/commit/90304602fcc4b2f61f8834861b71832d8f2c3135
  Author: Xiaozhu Meng 
  Date:   2018-07-23 (Mon, 23 Jul 2018)

  Changed paths:
M parseAPI/src/SymtabCodeSource.C

  Log Message:
  ---
  Should not parallelize SymtabCodeSource::init_hints, which basically
appending elements to a shared vector. Right now, almost all threads are
ideal to wait for lock to append.


Compare: https://github.com/dyninst/dyninst/compare/8323ae71e91c...90304602fcc4
  **NOTE:** This service been marked for deprecation: 
https://developer.github.com/changes/2018-04-25-github-services-deprecation/

  Functionality will be removed from GitHub.com on January 31st, 2019.
___
Dyninst-api mailing list
Dyninst-api@cs.wisc.edu
https://lists.cs.wisc.edu/mailman/listinfo/dyninst-api

[DynInst_API:] [PATCH] Eliminate aarch64 specific includes

2018-07-23 Thread William Cohen 
The use of /usr/include/bits/uio.h is questionable.  Anything in
/usr/include/bits is an internal header file and should not be used
directly by user code.  On Fedora 28 there isn't a
/usr/includes/bits/uio.h.  Also Eliminated the conditional includes to
simplify the code.

Signed-off-by: William Cohen 
---
 proccontrol/src/linux.C | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/proccontrol/src/linux.C b/proccontrol/src/linux.C
index de77ad0f4..7d102601d 100644
--- a/proccontrol/src/linux.C
+++ b/proccontrol/src/linux.C
@@ -69,12 +69,9 @@
 #include "boost/shared_ptr.hpp"
 
 //needed by GETREGSET/SETREGSET
-#if defined(arch_aarch64)
 #include
 #include
-#include
 #include
-#endif
 
 // Before glibc-2.7, sys/ptrace.h lacked PTRACE_O_* and PTRACE_EVENT_*, so we
 // need them from linux/ptrace.h.  (Conditionally, as later glibc conflicts.)
-- 
2.17.1

___
Dyninst-api mailing list
Dyninst-api@cs.wisc.edu
https://lists.cs.wisc.edu/mailman/listinfo/dyninst-api

[DynInst_API:] [PATCH] Avoid having initial declaration in 'for' loop (a C99 feature)

2018-07-23 Thread William Cohen 
Putting the initialization in the 'for' loop is a C99 language
specification feature.  GCC will flag its use as an error unless using
C99 mode.  For this particular case there is really nothing to be
gained by using it.  Moved the declaration to the beginning of the
function.

Signed-off-by: William Cohen 
---
 src/dyninst/test1.h | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/dyninst/test1.h b/src/dyninst/test1.h
index ac58c798..7e30ed01 100644
--- a/src/dyninst/test1.h
+++ b/src/dyninst/test1.h
@@ -34,7 +34,8 @@
 unsigned long get_pointer()
 {
 unsigned long val;
-for(short i = 0; i < sizeof(void *); i++) {
+short i;
+for(i = 0; i < sizeof(void *); i++) {
 val = (val << 8)|((i&0xFF)|0xA0);
 }
 return val;
-- 
2.17.1

___
Dyninst-api mailing list
Dyninst-api@cs.wisc.edu
https://lists.cs.wisc.edu/mailman/listinfo/dyninst-api

[DynInst_API:] [dyninst/dyninst] 6a7151: Fixing BaseTramp to save registers as it's done wi...

2018-07-23 Thread Sasha @leela 
  Branch: refs/heads/arm64/feature/relocation
  Home:   https://github.com/dyninst/dyninst
  Commit: 6a71517fb076390ef2c00b4df1dbc5b0607bb5fe
  
https://github.com/dyninst/dyninst/commit/6a71517fb076390ef2c00b4df1dbc5b0607bb5fe
  Author: Sasha @leela 
  Date:   2018-07-23 (Mon, 23 Jul 2018)

  Changed paths:
M dataflowAPI/src/ABI.C
M dyninstAPI/src/codegen-aarch64.C
M dyninstAPI/src/emit-aarch64.h
M dyninstAPI/src/inst-aarch64.C
M dyninstAPI/src/inst-aarch64.h

  Log Message:
  ---
  Fixing BaseTramp to save registers as it's done with PowerPC.

Now the BaseTramp reserves a fixed amount of memory for each stack frame,
and the registers are always saved in the same position, in order to be 
retrieved
if necessary for some functionality such as BPatch_paramExpr.

Test1_13 now passes after this modification.



  **NOTE:** This service been marked for deprecation: 
https://developer.github.com/changes/2018-04-25-github-services-deprecation/

  Functionality will be removed from GitHub.com on January 31st, 2019.
___
Dyninst-api mailing list
Dyninst-api@cs.wisc.edu
https://lists.cs.wisc.edu/mailman/listinfo/dyninst-api