[PATCH] D128848: Fix assertion when analyzing a for-loop with no condition expression

2022-06-29 Thread Eric Li via Phabricator via cfe-commits
li.zhe.hua abandoned this revision. li.zhe.hua added a comment. In D128848#3620185 , @ymandel wrote: > Looks like you and Stanislav had the same idea? > https://reviews.llvm.org/D128833 Seems like it. Repository: rG LLVM Github Monorepo CHANGES

[PATCH] D128848: Fix assertion when analyzing a for-loop with no condition expression

2022-06-29 Thread Yitzhak Mandelbaum via Phabricator via cfe-commits
ymandel added a comment. Looks like you and Stanislav had the same idea? https://reviews.llvm.org/D128833 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128848/new/ https://reviews.llvm.org/D128848 ___

[PATCH] D128848: Fix assertion when analyzing a for-loop with no condition expression

2022-06-29 Thread Eric Li via Phabricator via cfe-commits
li.zhe.hua created this revision. li.zhe.hua added a reviewer: ymandel. Herald added a project: All. li.zhe.hua requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. For loops are allowed to have an empty conditional expression; we should not