[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