MaskRay wrote:

> > @zygoloid Is reusing the message for regular stores clear (current 
> > behavior) enough?
> > ```
> > // CHECK-MEMCPY-STORE: misaligned.cpp:[[#@LINE+4]]{{(:12)?}}: runtime 
> > error: store to misaligned address [[PTR:0x[0-9a-f]*]] for type 'int *', 
> > which requires 4 byte alignment
> > ```
> 
> I predict we'll get a bug report saying "I didn't do a store, I used 
> `memcpy`, which is specified as doing a byte-by-byte copy". But given that 
> improving this would require adding a new entry point into the runtime 
> library, and we don't know how common or rare this situation will be, I think 
> it's probably OK to wait until someone complains.

Thanks. Let's wait until someone complains :)

https://github.com/llvm/llvm-project/pull/67766
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to