aeubanks added a comment.

Actually sorry, it does still seem like there are false positives on structs. 
Reduced:

  $ cat /tmp/a.cpp
  
  struct A {
    const char* a;
    const char* b;
    const char* c;
  };
  
  static constexpr A foo2 = A{"",
                              ""
                              "",
                              ""};
  
  $ ~/repos/llvm-project/build_cmake/bin/clang /tmp/a.cpp -o /dev/null -c 
-Wstring-concatenation
  /tmp/a.cpp:10:29: warning: suspicious concatenation of string literals in an 
array initialization; did you mean to separate the elements with a comma? 
[-Wstring-concatenation]
                              "",
                              ^
  /tmp/a.cpp:9:29: note: place parentheses around the string literal to silence 
warning
                              ""
                              ^
  1 warning generated.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D85545/new/

https://reviews.llvm.org/D85545

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to