[PATCH] D122431: Basic support for posix_memalign / __builtin_object_size interaction

[Dávid Bolvanský via Phabricator via llvm-commits]

xbolva00 added inline comments.


================
Comment at: llvm/lib/Analysis/MemoryBuiltins.cpp:899
+                                        ConstantInt::get(CB->getType(), 0));
+      Optional<bool> Checked = isImpliedByDomCondition(FakeCond, &Load, DL);
+      delete FakeCond;
----------------
Do you need FakeCond at all?

llvm::isImpliedByDomCondition (CmpInst::Predicate Pred, const Value *LHS, const Value *RHS, const Instruction *ContextI, const DataLayout &DL)


================
Comment at: llvm/lib/Analysis/MemoryBuiltins.cpp:156
     {LibFunc___kmpc_alloc_shared,               {MallocLike,       1,  0, -1, -1, MallocFamily::KmpcAllocShared}},
-    // TODO: Handle "int posix_memalign(void **, size_t, size_t)"
 };
----------------
xbolva00 wrote:
> why remove? no record above for posix_memalign was added..
Still no answer


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

https://reviews.llvm.org/D122431