[PATCH] D47857: [RISCV] Add machine function pass to merge base + offset
Alex Bradbury via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 13 17:09:25 PDT 2018
asb added inline comments.
================
Comment at: lib/Target/RISCV/RISCVMergeBaseOffset.cpp:1
+//== RISCVMergeBaseOffset.cpp - Fold offset into address lowering sequence ==//
+// The LLVM Compiler Infrastructure
----------------
asb wrote:
> This line is only 79 chars long. Probably nicer to keep the usual format (three =) and have a shorter description.
>
> e.g. `//== RISCVMergeBaseOffset.cpp - Fold offset into address lowering sequence ==//`
Oops, wrong paste in that comment. Corrected:
`//===-- RISCVMergeBaseOffset.cpp - Optimise address calculations ---------===//`
https://reviews.llvm.org/D47857
More information about the llvm-commits
mailing list