[PATCH] D126903: [clang] Add support for __builtin_memset_inline

Clement Courbet via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 26 01:52:03 PDT 2022


courbet added a comment.

I think there's a bug in the code though:

  // If this is memset, we just need to see if the offset is valid in the size
    // of the memset..
    if (MI->getIntrinsicID() == Intrinsic::memset)

should really be:

  // If this is memset, we just need to see if the offset is valid in the size
    // of the memset..
    if (isa<MemSetInst>(MI))




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126903



More information about the cfe-commits mailing list