[PATCH] D133648: Clang, increase upper bound of partially initialized array sizes

2022-09-25 Thread Ofek Shochat via Phabricator via cfe-commits
OfekShochat marked an inline comment as done. OfekShochat added a comment. @efriedma would you be able to commit this? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D133648/new/ https://reviews.llvm.org/D133648 _

[PATCH] D133648: Clang, increase upper bound of partially initialized array sizes

2022-09-14 Thread Ofek Shochat via Phabricator via cfe-commits
OfekShochat marked an inline comment as done. OfekShochat added a comment. I created a diff for the AsmPrinter part, its at D133845 Comment at: clang/lib/Sema/SemaInit.cpp:866 ElementEntity.getKind() == InitializedEntity::EK_VectorEl

[PATCH] D133648: Clang, increase upper bound of partially initialized array sizes

2022-09-12 Thread Ofek Shochat via Phabricator via cfe-commits
OfekShochat updated this revision to Diff 459642. OfekShochat added a comment. Clang, increase upper bound of partially initialized array sizes fixes issue with emitting partially initialized constant arrays larger than 2^32. issue #57353 on github. Repository: rG LLVM Github Monorepo CHANG

[PATCH] D133648: Clang, increase upper bound of partially initialized array sizes

2022-09-12 Thread Eli Friedman via Phabricator via cfe-commits
efriedma added inline comments. Comment at: clang/lib/Sema/SemaInit.cpp:866 ElementEntity.getKind() == InitializedEntity::EK_VectorElement) ElementEntity.setElementIndex(Init); shafik wrote: > `setElementIndex(...)` takes `unsigned` as well and

[PATCH] D133648: Clang, increase upper bound of partially initialized array sizes

2022-09-12 Thread Shafik Yaghmour via Phabricator via cfe-commits
shafik added inline comments. Comment at: clang/lib/Sema/SemaInit.cpp:866 ElementEntity.getKind() == InitializedEntity::EK_VectorElement) ElementEntity.setElementIndex(Init); `setElementIndex(...)` takes `unsigned` as well and therefore `Initia