ChuanqiXu added a comment.

> which should be correct. It is implemented by 
> CodeGenFunction::EmitBuiltinAlignTo.

I agree it is correct. I just want to say we should comment it to avoid 
confusing.

Since the patch could handle the case if the frontend tries to search 
`::operator new(size_t, align_val_t)`, this patch should be based on D102147 
<https://reviews.llvm.org/D102147>.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97915

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

Reply via email to