[PATCH] D102452: [Remarks] Add analysis remarks for memset/memcpy/memmove lengths

[Vlad Vereschaka via Phabricator via llvm-commits]

vvereschaka added a comment.

Hi @jroelofs,

we've started getting the warnings for newly created file `llvm/lib/Transforms/Utils/MemoryOpRemark.cpp` on the cross toolchain builders:

  c:\buildbot\as-builder-2\x-aarch64\llvm-project\llvm\lib\transforms\utils\memoryopremark.cpp(123) : warning C4715: 'llvm::MemoryOpRemark::remarkName': not all control paths return a value
  c:\buildbot\as-builder-2\x-aarch64\llvm-project\llvm\lib\transforms\utils\memoryopremark.cpp(381) : warning C4715: 'llvm::AutoInitRemark::remarkName': not all control paths return a value

one of the builds as example: https://lab.llvm.org/buildbot/#/builders/119/builds/3889

there is no return for the method's argument unknown value.

Would you take care of it?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102452