[PATCH] D22388: MIRParser: Use shorter cfi identifiers

Matthias Braun via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 14 16:36:27 PDT 2016


MatzeB created this revision.
MatzeB added reviewers: qcolombet, arphaman.
MatzeB added a subscriber: llvm-commits.
MatzeB set the repository for this revision to rL LLVM.
Herald added subscribers: mcrosier, dsanders.

In an instruction like:
```
    CFI_INSTRUCTION .cfi_def_cfa ...
```
we can drop the '.cfi_' prefix since that should be obvious by the
context:
```
    CFI_INSTRUCTION def_cfa ...
```

While being a terser and cleaner syntax this will enable upcoming changes that re-use the '.' symbol as a separate token rather than making it part of an identifier by default.

Repository:
  rL LLVM

https://reviews.llvm.org/D22388

Files:
  lib/CodeGen/MIRParser/MILexer.cpp
  lib/CodeGen/MIRPrinter.cpp
  test/CodeGen/MIR/AArch64/cfi-def-cfa.mir
  test/CodeGen/MIR/ARM/ARMLoadStoreDBG.mir
  test/CodeGen/MIR/ARM/cfi-same-value.mir
  test/CodeGen/MIR/ARM/sched-it-debug-nodes.mir
  test/CodeGen/MIR/Mips/memory-operands.mir
  test/CodeGen/MIR/X86/cfi-def-cfa-offset.mir
  test/CodeGen/MIR/X86/cfi-def-cfa-register.mir
  test/CodeGen/MIR/X86/cfi-offset.mir
  test/CodeGen/MIR/X86/early-clobber-register-flag.mir
  test/CodeGen/MIR/X86/expected-comma-after-cfi-register.mir
  test/CodeGen/MIR/X86/expected-offset-after-cfi-operand.mir
  test/CodeGen/MIR/X86/expected-register-after-cfi-operand.mir
  test/CodeGen/MIR/X86/fixed-stack-memory-operands.mir
  test/CodeGen/MIR/X86/large-cfi-offset-number-error.mir
  test/CodeGen/MIR/X86/liveout-register-mask.mir
  test/CodeGen/MIR/X86/memory-operands.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D22388.64064.patch
Type: text/x-patch
Size: 17646 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160714/f4f47283/attachment.bin>


More information about the llvm-commits mailing list