ioeric added a comment.
Still LGTM. Please run `ninja check-clang` and fix the warning. I'll land this
patch for you afterwards.
================
Comment at: lib/Tooling/RefactoringCallbacks.cpp:222
+ }
+ default:
+ llvm_unreachable("Element.Type not recognized");
----------------
```
ninja check-clang
RefactoringCallbacks.cpp:222:5: warning: default label in switch which covers
all enumeration values [-Wcovered-switch-default]
```
Please remove the `default`.
https://reviews.llvm.org/D29621
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
