Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
LGTM, but see a few inline comments.
| llvm/trunk/docs/ProgrammersManual.rst | ||
|---|---|---|
| 1215 | Somehow I feel the word bisect should be present in this description, that may help the reader to "connect" quickly to the use case. | |
| 1233 | I don't know why this block is indented (I don't write rst every day either). If it is intentional then fine :) | |
| 1265 | The line returns seem a bit funky? (80 cols) | |
| llvm/trunk/docs/ProgrammersManual.rst | ||
|---|---|---|
| 1233 | I copied the style from what we did in statistics above, but looking at the output of that and this, it's clearly wrong. | |
Somehow I feel the word bisect should be present in this description, that may help the reader to "connect" quickly to the use case.