melver added inline comments.
================ Comment at: llvm/lib/CodeGen/SanitizerBinaryMetadata.cpp:57 + // Assume it currently only has features. + assert(AuxMDs.size() == 1); + auto *Features = cast<ConstantAsMetadata>(AuxMDs.getOperand(0))->getValue(); ---------------- Probably should be getNumOperands()? https://reviews.llvm.org/rGdbb11309665fdf7d411d25cc8f2e6a0c8f658143 `-DLLVM_ENABLE_ASSERTIONS=On` will catch it. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136078/new/ https://reviews.llvm.org/D136078 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
