This revision was automatically updated to reflect the committed changes.
Closed by commit rG206a66de5902: Sanitizers.h - remove MathExtras.h include
dependency (authored by RKSimon).
Changed prior to commit:
https://reviews.llvm.org/D103953?vs=350845&id=350882#toc
Repository:
rG LLVM Github
pgousseau accepted this revision.
pgousseau added a comment.
This revision is now accepted and ready to land.
LGTM!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D103953/new/
https://reviews.llvm.org/D103953
RKSimon added inline comments.
Comment at: clang/lib/CodeGen/CGExpr.cpp:3129
static CheckRecoverableKind getRecoverableKind(SanitizerMask Kind) {
- assert(Kind.countPopulation() == 1);
+ assert(Kind.isPowerOf2());
if (Kind == SanitizerKind::Function || Kind == SanitizerKind
RKSimon created this revision.
RKSimon added reviewers: pgousseau, filcab, bkramer.
Herald added a subscriber: dexonsmith.
RKSimon requested review of this revision.
Herald added a project: clang.
We include the MathExtras.h header purely for the countPopulation() method - by
moving this into San
