[PATCH] D63663: [clang-doc] Add html links to references

Julie Hockett via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 9 14:57:39 PDT 2019


juliehockett added a comment.

I've also just noticed that the YAML generator is missing from this -- could you add the `Path` fields to hat output as well?



================
Comment at: clang-tools-extra/clang-doc/HTMLGenerator.cpp:213
+  llvm::SmallString<128> Path = Type.Path;
+  llvm::sys::path::append(Path, Type.Name + ".html");
+  return genLink(Type.Name, Path);
----------------
Now that the paths aren't absolute, you'll need to make this relative to the current path (something like https://github.com/llvm/llvm-project/blob/master/clang/lib/Lex/PPLexerChange.cpp#L201)


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

https://reviews.llvm.org/D63663





More information about the cfe-commits mailing list