[PATCH] D143128: [-Wunsafe-buffer-usage][WIP] Fix-Its transforming `&DRE[any]` to `DRE.data() + any`

Ziqing Luo via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 6 16:30:54 PST 2023


ziqingluo-90 added a comment.

In D143128#4108375 <https://reviews.llvm.org/D143128#4108375>, @NoQ wrote:

> Why do we prefer `DRE.data() + any` to `&DRE.data()[any]`? It could be much less intrusive this way, and the safety guarantees are the same.

It is actually `(DRE.data() + any)` versus `&DRE.data()[any]`. Are they quite the same in terms of being intrusive?


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

https://reviews.llvm.org/D143128



More information about the cfe-commits mailing list