aaron.ballman added a comment.

Thank you for this cleanup! In general, I thin this looks correct. However, I 
know we've had to fix a bunch of options that cause the sphinx build to fail 
(IIRC, oftentimes due to duplicate options) and our precommit CI doesn't test 
the documentation build. Did you try building the docs locally to ensure there 
are no new warnings/errors from Sphinx?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138088

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

Reply via email to