================
Comment at: lib/Transforms/Instrumentation/AddressSanitizer.cpp:1359
@@ -1349,3 +1358,3 @@
bool Changed = false;
----------------
samsonov wrote:
> Wait a second.... Do you do *anything* in AddressSanitizerModule pass in
> `CompileKernel` mode? You don't add module constructor, and you don't
> instrument globals. Maybe, we should just avoid creating this pass in the
> first place?
Or perhaps instrumentation of globals will follow shortly so it's ok to have
this pass at hand.
================
Comment at: lib/Transforms/Instrumentation/AddressSanitizer.cpp:1379
@@ -1368,2 +1378,3 @@
const std::string ExpStr = Exp ? "exp_" : "";
+ const std::string SuffixStr = CompileKernel ? "N" : "_n";
const Type *ExpType = Exp ? Type::getInt32Ty(*C) : nullptr;
----------------
Perhaps use "_noabort" suffix to match gcc?
http://reviews.llvm.org/D10411
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
_______________________________________________
cfe-commits mailing list
cfe-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
