amyk added a comment.
Overall this change looks good to me, although I do have one question.
================
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:15208
- for (unsigned i = 0, e = E->getNumArgs(); i != e; i++) {
- if (E->getArg(i)->getType()->isArrayType())
- Ops.push_back(EmitArrayToPointerDecay(E->getArg(i)).getPointer());
----------------
A question I have is do we not need to consider
this/`EmitArrayToPointerDecay()` anymore? Was this not used for anything?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D121637/new/
https://reviews.llvm.org/D121637
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
