[PATCH] D62214: Remove extra if case.

2019-05-22 Thread Amy Huang via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL361400: Combine two if cases because the second one is never reached. (authored by akhuang, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit:

[PATCH] D62214: Remove extra if case.

2019-05-21 Thread Reid Kleckner via Phabricator via cfe-commits
rnk accepted this revision. rnk added a comment. This revision is now accepted and ready to land. lgtm Testing offline showed that it was impossible to reach the return with a real global variable with an enumeration type. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D62214: Remove extra if case.

2019-05-21 Thread Amy Huang via Phabricator via cfe-commits
akhuang created this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D62214 Files: clang/lib/CodeGen/CGDebugInfo.cpp Index: clang/lib/CodeGen/CGDebugInfo.cpp