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

[Clement Courbet via Phabricator via cfe-commits]

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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits