MaskRay added a comment.

The CPython usage is a real UB: https://github.com/python/cpython/issues/94250

In D126864#3609497 <https://reviews.llvm.org/D126864#3609497>, @efriedma wrote:

> I suspect the refactoring in the latest version of the patch accidentally 
> made UBSan a bit more strict by default.

Yes, the patch accidentally dropped a -fsanitize=array-bounds workaround for 
size-1 array as the last member of a structure. I added it back in 
572b08790a69f955ae0cbb1b4a7d4a215f15dad9 
<https://reviews.llvm.org/rG572b08790a69f955ae0cbb1b4a7d4a215f15dad9>


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

https://reviews.llvm.org/D126864

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

Reply via email to