[PATCH] D136078: Use-after-return sanitizer binary metadata

Marco Elver via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 29 09:09:39 PST 2022


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



More information about the cfe-commits mailing list