kkleine added a comment.

@njames93 I could do that but the original Macros had were defined without a 
semicolon at the end and one had to add it manually. See this revision in which 
I replaced some occurrences of `DISALLOW_COPY_AND_ASSIGN`: 
eaebcbc67926a18befaa297f1778edde63baec9b 
<https://reviews.llvm.org/rGeaebcbc67926a18befaa297f1778edde63baec9b>. What do 
you suggest? Keep the semicolon to more closely match the original macros or 
remove it to make the test happy?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80531

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

Reply via email to