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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
