[PATCH] [RFC] Print exact source locations of global variables in ASan error reports.
Alexander Potapenko
glider at google.com
Fri Jun 20 06:31:21 PDT 2014
AddressSanitizer.cpp LGTM
================
Comment at: lib/Transforms/Instrumentation/AddressSanitizer.cpp:1069
@@ -1004,2 +1068,3 @@
GlobalVariable *G = GlobalsToChange[i];
+
PointerType *PtrTy = cast<PointerType>(G->getType());
----------------
nit: spare newline?
================
Comment at: projects/compiler-rt/test/asan/TestCases/global-location.cc:11
@@ +10,3 @@
+
+class C {
+ public:
----------------
"struct C", maybe?
================
Comment at: projects/compiler-rt/test/asan/TestCases/global-location.cc:17
@@ +16,3 @@
+int global[10];
+// GLOB: 0x{{.*}} is located 4 bytes to the right of global variable 'global' defined in '{{.*}}/global-location.cc:[[@LINE-1]]:5' {{.*}} of size 40
+int C::array[10];
----------------
Do you need the slash before global-location.cc? Isn't it possible that there's no directory prefix (maybe not in the current setup)?
http://reviews.llvm.org/D4203
More information about the cfe-commits
mailing list