 Chromium Code Reviews
 Chromium Code Reviews Issue 2042063002:
  Created Ice::Instrumentation base class and accompanying hooks.  (Closed) 
  Base URL: https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
    
  
    Issue 2042063002:
  Created Ice::Instrumentation base class and accompanying hooks.  (Closed) 
  Base URL: https://chromium.googlesource.com/native_client/pnacl-subzero.git@master| Index: src/IceCfg.cpp | 
| diff --git a/src/IceCfg.cpp b/src/IceCfg.cpp | 
| index 8f6a2fffaca0ecbf8d040b895458c0b33bb69895..a7595ff0e782cdd7832beb566f90e2872c08fec3 100644 | 
| --- a/src/IceCfg.cpp | 
| +++ b/src/IceCfg.cpp | 
| @@ -22,6 +22,7 @@ | 
| #include "IceELFObjectWriter.h" | 
| #include "IceGlobalInits.h" | 
| #include "IceInst.h" | 
| +#include "IceInstrumentation.h" | 
| #include "IceInstVarIter.h" | 
| #include "IceLiveness.h" | 
| #include "IceLoopAnalyzer.h" | 
| @@ -238,6 +239,12 @@ void Cfg::translate() { | 
| if (auto *Var64On32 = llvm::dyn_cast<Variable64On32>(Var)) | 
| Var64On32->initHiLo(this); | 
| + // Instrument the Cfg, e.g. with AddressSanitizer | 
| + if (!BuildDefs::minimal()) { | 
| 
Karl
2016/06/07 14:42:40
Shouldn't these also be under a CL flag for saniti
 
tlively
2016/06/07 18:14:41
Done.
 | 
| + getContext()->instrumentFunc(this); | 
| + dump("Instrumented CFG"); | 
| + } | 
| + | 
| // The set of translation passes and their order are determined by the | 
| // target. | 
| getTarget()->translate(); |