[PATCH] D31266: [sancov] fixing too aggressive instrumentation elimination

Kostya Serebryany via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 23 16:38:10 PDT 2017


kcc added inline comments.


================
Comment at: lib/Transforms/Instrumentation/SanitizerCoverage.cpp:422
   }
 
   return true;
----------------
what if you add here something like this: 
 for (const BasicBlock *SUCC : make_range(succ_begin(BB), succ_end(BB))) {
    if (PDT->dominates(SUCC, BB))
      return false;
  }


https://reviews.llvm.org/D31266





More information about the llvm-commits mailing list