[clang-tools-extra] [clang-doc] Wire Markdown parser into MDGenerator for fenced code blocks (PR #201746)

Neil Nair via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 4 23:13:01 PDT 2026


Neil-N4 wrote:

> It's pretty hard to review code like this since this contains all the changes present in the other PR, so we can't distinguish the diffs since they're both against main. Technically this isn't stacked on #200302 since it's not being compared against that branch.
> 
> LLVM has a couple ways to do stacked PRs. There's spr (which I never used thanks to Paul's warnings) and [Graphite](https://llvm.org/docs/GitHub.html#using-graphite-for-stacked-pull-requests). If you want to keep your work compartmentalized I suggest learning it. It's pretty helpful.
> 
> You could also just compare this branch against your first one but I don't think that scales well.

Thanks for the tip. For now I'll keep the PR as-is since it's a draft and we're still iterating. Happy to set up Graphite before the next round if that makes review easier. Will look into it before the call.

https://github.com/llvm/llvm-project/pull/201746


More information about the cfe-commits mailing list