carlosgalvezp added a comment.

Thanks for the clarification, makes sense! I got the notification about the 
build failure, thanks for fixing. I actually added it in the first place but 
then noticed that checkers typically didn't include any STL header so I thought 
maybe there was a restriction on their usage or something. The code compiled 
just fine when running the "clang-check-tools" target. Good to know for the 
next time :)

PS: I've also reported the build errors in pre-merge, maybe they would have 
caught it earlier.


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

https://reviews.llvm.org/D110493

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

Reply via email to