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

[Gabor Marton via Phabricator via cfe-commits]

martong added inline comments.


================
Comment at: clang/utils/TableGen/ClangSACheckersEmitter.cpp:90-91
+
+  return (llvm::Twine("https://clang.llvm.org/docs/analyzer/checkers.html#") +
+          CheckerFullName)
       .str();
----------------
> This patch will ensure that the doc urls produced by tablegen for the ClangSA, will use the new url. Nothing else will be changed.

If that was the case then we'd need only this hunk of change plus the test file. Perhaps it would make sense to either split this to more patches
- Adding `Sep`
- Handling missing documentation
- new url
Or please change the description of the patch about these changes.


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

https://reviews.llvm.org/D121387