[PATCH] D145463: [Inliner][NFC] Remove redundant nullptr check

Mircea Trofin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 7 09:51:25 PST 2023


mtrofin added a comment.

In D145463#4175673 <https://reviews.llvm.org/D145463#4175673>, @AlexM wrote:

> I'm very new to this and trying to get my toes wet in the review process with some very minor non-controversial changes. Please let me know if I've made any mistakes in the process.

Would adding an `assert(Callee && "Expected to be non-null due to check at the start of the loop")` (or something like that) above the `if`-check make Coverity happy?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D145463/new/

https://reviews.llvm.org/D145463



More information about the llvm-commits mailing list