[PATCH] D121387: [analyzer] ClangSA should tablegen doc urls refering to the main doc page

[Denys Petrov via Phabricator via cfe-commits]

ASDenysPetrov accepted this revision.
ASDenysPetrov added a comment.

Thanks! LGTM.



================
Comment at: clang/utils/TableGen/ClangSACheckersEmitter.cpp:87-90
+  std::string CheckerFullName = StringRef(getCheckerFullName(&R, "-")).lower();
+  return (llvm::Twine("https://clang.llvm.org/docs/analyzer/checkers.html#";) +
+          CheckerFullName)
       .str();
----------------
IMO it's a bit messy here.


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

https://reviews.llvm.org/D121387

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