pt: Update PIN to v3.11-97998

Fraunhofer-AISEC · Mar 3, 2023 · 1e4641b · 1e4641b
1 parent af8fb5f
commit 1e4641b
Show file tree

Hide file tree

Showing 5 changed files with 20 additions and 24 deletions.
diff --git a/data-gui b/data-gui
diff --git a/pin/Makefile b/pin/Makefile
@@ -28,7 +28,6 @@ PROJ := DATA - Pin
 
 # Pin Version and Source
 
-#VER_STRING=pin-3.7-97619-g0d0c92f4f-gcc-linux
 VER_STRING=pin-3.11-97998-g7ecce2dac-gcc-linux
 
 TAR=$(VER_STRING).tar.gz

diff --git a/pintool/addrtrace.cpp b/pintool/addrtrace.cpp
@@ -26,6 +26,7 @@
 /***********************************************************************/
 
 #include "pin.H"
+#include "pin-macros.H"
 #include <fcntl.h>
 #include <fstream>
 #include <getopt.h>
@@ -42,25 +43,6 @@
 #include <unistd.h>
 #include <vector>
 
-/**
- * Pin 3.11 Documentation:
- * https://software.intel.com/sites/landingpage/pintool/docs/97998/Pin/html
- */
-
-// Pin above 3.7.97720 deprecates some functions
-#if (PIN_PRODUCT_VERSION_MAJOR > 3) ||                                         \
-    (PIN_PRODUCT_VERSION_MAJOR == 3 && PIN_PRODUCT_VERSION_MINOR > 7) ||       \
-    (PIN_PRODUCT_VERSION_MAJOR == 3 && PIN_PRODUCT_VERSION_MINOR == 7 &&       \
-     PIN_BUILD_NUMBER > 97720)
-#define INS_IS_INDIRECT INS_IsIndirectControlFlow
-#define INS_HAS_TAKEN_BRANCH INS_IsValidForIpointTakenBranch
-#define INS_HAS_IPOINT_AFTER INS_IsValidForIpointAfter
-#else
-#define INS_IS_INDIRECT INS_IsIndirectBranchOrCall
-#define INS_HAS_TAKEN_BRANCH INS_IsBranchOrCall
-#define INS_HAS_IPOINT_AFTER INS_HasFallThrough
-#endif
-
 using namespace std;
 
 /***********************************************************************/

diff --git a/pintool/call-stack.cpp b/pintool/call-stack.cpp
@@ -30,6 +30,7 @@ OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 END_LEGAL */
 
 #include "call-stack.H"
+#include "pin-macros.H"
 #include <iomanip>
 #include <iostream>
 #include <sstream>
@@ -113,9 +114,9 @@ static void i_trace(TRACE trace, void *v) {
             continue;
         }
 #endif
-        if (INS_IsDirectBranchOrCall(tail)) {
+        if (INS_IS_DIRECT(tail)) {
             // check if direct or indirect call and take the target accordingly
-            ADDRINT target = INS_DirectBranchOrCallTargetAddress(tail);
+            ADDRINT target = INS_DIRECT(tail);
             INS_InsertCall(tail, IPOINT_BEFORE, (AFUNPTR)a_process_call,
                            IARG_ADDRINT, target, IARG_REG_VALUE, REG_STACK_PTR,
                            IARG_REG_VALUE, vreg, IARG_END);
@@ -127,7 +128,7 @@ static void i_trace(TRACE trace, void *v) {
                                IARG_END);
             }
         }
-        if (INS_IsIndirectBranchOrCall(tail) && !INS_IsRet(tail)) {
+        if (INS_IS_INDIRECT(tail) && !INS_IsRet(tail)) {
             INS_InsertCall(tail, IPOINT_TAKEN_BRANCH, (AFUNPTR)a_process_call,
                            IARG_BRANCH_TARGET_ADDR, IARG_REG_VALUE,
                            REG_STACK_PTR, IARG_REG_VALUE, vreg, IARG_END);

diff --git a/pintool/pin-macros.H b/pintool/pin-macros.H
@@ -0,0 +1,14 @@
+#if !defined(PIN_MACROS_H_)
+#define PIN_MACROS_H_
+
+/**
+ * We support Pin 3.11-97998 Documentation:
+ * https://software.intel.com/sites/landingpage/pintool/docs/97998/Pin/html
+ */
+#define INS_DIRECT INS_DirectControlFlowTargetAddress
+#define INS_IS_DIRECT INS_IsDirectControlFlow
+#define INS_IS_INDIRECT INS_IsIndirectControlFlow
+#define INS_HAS_TAKEN_BRANCH INS_IsValidForIpointTakenBranch
+#define INS_HAS_IPOINT_AFTER INS_IsValidForIpointAfter
+
+#endif