jkorous added a comment.
Sorry for the delay - I was busy with other things for past two weeks.
================
Comment at: Sema/SemaExpr.cpp:5338
Context.DependentTy, VK_RValue, RParenLoc);
} else {
----------------
JDevlieghere wrote:
> While you're at it you might as well remove the else branch here.
Sorry, I am not following. Do you mean just refactoring?
```
if a
foo
else
bar
```
->
```
if a
foo
bar
```
Or do you really mean removing the else branch? I don't see how that would be
NFC.
Repository:
rC Clang
https://reviews.llvm.org/D51488
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
