[PATCH] D23561: [RISCV 4/10] Add basic RISCV{InstrFormats, InstrInfo, RegisterInfo, }.td

Tim Northover via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 22 09:18:48 PDT 2016


On 22 August 2016 at 02:57, Alex Bradbury <asb at asbradbury.org> wrote:
> Do you have any thoughts on a clearer naming convention?

Another possibility is a more semantically meaningful name like
"jaltarget", if we're going to try to describe the output range. But
of the 3 you gave I'm mildly swayed by your symmetry argument. I think
the most important point is that there's some difference so people
know something weird is going on and can look up the details.

Tim.


More information about the llvm-commits mailing list