[PATCH] D131657: Remove redundant condition check, NFC

2022-08-11 Thread Jun Zhang via Phabricator via cfe-commits
junaire added a comment. In D131657#3715976 , @erichkeane wrote: > LGTM! Let us know if you need this committed for you. Thanks! But I already have commit access :^) Closed in efc75a2baedc7405193e3e0f5ea9aaa881783cec

[PATCH] D131657: Remove redundant condition check, NFC

2022-08-11 Thread Erich Keane via Phabricator via cfe-commits
erichkeane accepted this revision. erichkeane added a comment. LGTM! Let us know if you need this committed for you. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131657/new/ https://reviews.llvm.org/D131657 __

[PATCH] D131657: Remove redundant condition check, NFC

2022-08-11 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LTGM! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131657/new/ https://reviews.llvm.org/D131657

[PATCH] D131657: Remove redundant condition check, NFC

2022-08-11 Thread Jun Zhang via Phabricator via cfe-commits
junaire created this revision. Herald added a project: All. junaire requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Signed-off-by: Jun Zhang Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D131657 Files: clang/lib/Pars