[llvm] be2227b - [hwasan][test] Rework memaccess-clobber.ll

Arthur Eubanks via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 10 09:14:20 PST 2022


Author: Arthur Eubanks
Date: 2022-02-10T09:12:18-08:00
New Revision: be2227b6c7aff4851b30e3fb089c4531f6e4677f

URL: https://github.com/llvm/llvm-project/commit/be2227b6c7aff4851b30e3fb089c4531f6e4677f
DIFF: https://github.com/llvm/llvm-project/commit/be2227b6c7aff4851b30e3fb089c4531f6e4677f.diff

LOG: [hwasan][test] Rework memaccess-clobber.ll

Previously memaccess-clobber.ll relied on both legacy PM-specific things
like `-analyze` and MemoryDependenceAnalysis, which are both deprecated.

This uses MemorySSA, which is the cool new thing that a bunch of passes
have migrated to.

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

Added: 
    

Modified: 
    llvm/test/Instrumentation/HWAddressSanitizer/memaccess-clobber.ll

Removed: 
    


################################################################################
diff  --git a/llvm/test/Instrumentation/HWAddressSanitizer/memaccess-clobber.ll b/llvm/test/Instrumentation/HWAddressSanitizer/memaccess-clobber.ll
index dc3f964d6ef2e..3704da6b0835d 100644
--- a/llvm/test/Instrumentation/HWAddressSanitizer/memaccess-clobber.ll
+++ b/llvm/test/Instrumentation/HWAddressSanitizer/memaccess-clobber.ll
@@ -1,6 +1,6 @@
 ; Make sure memaccess checks preceed the following reads.
 ;
-; RUN: opt < %s -S -enable-new-pm=0 -hwasan -hwasan-use-stack-safety=0 -basic-aa -memdep -print-memdeps -analyze -mtriple aarch64-linux-android30 | FileCheck %s
+; RUN: opt < %s -disable-output 2>&1 -passes='hwasan,print<memoryssa>' -hwasan-use-stack-safety=0 -mtriple aarch64-linux-android30 | FileCheck %s
 
 target datalayout = "e-m:e-i8:8:32-i16:16:32-i64:64-i128:128-n32:64-S128"
 target triple = "aarch64--linux-android10000"
@@ -10,11 +10,14 @@ declare void @use32(i32*)
 define i32 @test_alloca() sanitize_hwaddress {
 entry:
   %x = alloca i32, align 4
+  ; CHECK: call void @use32
   call void @use32(i32* nonnull %x)
-  ; CHECK: Clobber from:   call void @llvm.hwasan.check.memaccess.shortgranule
-  ; CHECK-NEXT: load i32, i32* %x.hwasan, align 4
+  ; CHECK: [[A:[0-9]+]] = MemoryDef({{[0-9]+}})
+  ; CHECK-NEXT: call void @llvm.hwasan.check.memaccess.shortgranules
+  ; CHECK: MemoryUse([[A]])
+  ; CHECK-NEXT: load i32, i32* %x.hwasan
   %y = load i32, i32* %x
-  ; CHECK:  Clobber from:   %y = load i32, i32* %x.hwasan, align 4
-  ; CHECK-NEXT: call void @llvm.memset.p0i8.i64(i8* align 1 {{.*}}, i8 0, i64 1, i1 false)
+  ; CHECK: {{[0-9]+}} = MemoryDef([[A]])
+  ; CHECK-NEXT: call void @llvm.memset.p0i8.i64
   ret i32 %y
 }


        


More information about the llvm-commits mailing list