[compiler-rt] r288729 - [XRay][AArch64] Attempt to fix unstable test XRay-aarch64-linux::patching-unpatching.cc

Serge Rogatch via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 5 15:29:56 PST 2016


Author: rserge
Date: Mon Dec  5 17:29:56 2016
New Revision: 288729

URL: http://llvm.org/viewvc/llvm-project?rev=288729&view=rev
Log:
[XRay][AArch64] Attempt to fix unstable test XRay-aarch64-linux::patching-unpatching.cc

Summary: Currently test XRay-aarch64-linux::patching-unpatching.cc sometimes passes, sometimes fails. This is an attempt to fix it by handling better the situations when both `__arm__` and `__aarch64__` are defined.

Reviewers: dberris, rengolin

Subscribers: llvm-commits, iid_iunknown, aemerson, rengolin, dberris

Differential Revision: https://reviews.llvm.org/D27421

Modified:
    compiler-rt/trunk/lib/xray/xray_interface.cc

Modified: compiler-rt/trunk/lib/xray/xray_interface.cc
URL: http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/xray/xray_interface.cc?rev=288729&r1=288728&r2=288729&view=diff
==============================================================================
--- compiler-rt/trunk/lib/xray/xray_interface.cc (original)
+++ compiler-rt/trunk/lib/xray/xray_interface.cc Mon Dec  5 17:29:56 2016
@@ -31,10 +31,10 @@ namespace __xray {
 // FIXME: The actual length is 11 bytes. Why was length 12 passed to mprotect()
 // ?
 static const int16_t cSledLength = 12;
-#elif defined(__arm__)
-static const int16_t cSledLength = 28;
 #elif defined(__aarch64__)
 static const int16_t cSledLength = 32;
+#elif defined(__arm__)
+static const int16_t cSledLength = 28;
 #else
 #error "Unsupported CPU Architecture"
 #endif /* CPU architecture */




More information about the llvm-commits mailing list