This is an archive of the discontinued LLVM Phabricator instance.

MIRParser: Use shorter cfi identifiers
ClosedPublic

Authored by MatzeB on Jul 14 2016, 4:36 PM.

Details

Summary

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.

Diff Detail

Repository
rL LLVM

Event Timeline

MatzeB updated this revision to Diff 64064.Jul 14 2016, 4:36 PM
MatzeB retitled this revision from to MIRParser: Use shorter cfi identifiers.
MatzeB updated this object.
MatzeB added reviewers: qcolombet, arphaman.
MatzeB set the repository for this revision to rL LLVM.
MatzeB added a subscriber: llvm-commits.
qcolombet accepted this revision.Jul 22 2016, 4:04 PM
qcolombet edited edge metadata.

LGTM.

This revision is now accepted and ready to land.Jul 22 2016, 4:04 PM
This revision was automatically updated to reflect the committed changes.