__builtin_verbose_trap with empty string arguments should be treated as an error

Clang currently accepts this:

```
void test(void) {
__builtin_verbose_trap("","");
}
```

Clang's sema should not accept this as it doesn't make any sense to have both the category and message parameters be empty.