Page MenuHomePhabricator

[Alignment] Use llvm::Align in MachineFunction and TargetLowering - fixes mir parsing

Authored by gchatelet on Wed, Sep 11, 2:41 AM.



This catches malformed mir files which specify alignment as log2 instead of pow2.
See for reference,

Unit tests are automatically fixed with a custom tool, 3 needed special care:

  • llvm/test/CodeGen/MIR/Generic/machine-function.mir
  • llvm/test/CodeGen/Mips/unaligned-memops-mapping.mir
  • llvm/test/DebugInfo/X86/debug-loc-offset.mir

This is patch is part of a series to introduce an Alignment type.
See this thread for context:
See this patch for the introduction of the type:

Diff Detail


Event Timeline

gchatelet created this revision.Wed, Sep 11, 2:41 AM
Herald added a project: Restricted Project. · View Herald TranscriptWed, Sep 11, 2:41 AM

Can you point to the tests that have anything interesting to look at ?

37 ↗(On Diff #219676)

no s

gchatelet edited the summary of this revision. (Show Details)Wed, Sep 11, 3:49 AM
gchatelet updated this revision to Diff 219684.Wed, Sep 11, 3:51 AM
  • Address comments
courbet accepted this revision.Wed, Sep 11, 3:51 AM
This revision is now accepted and ready to land.Wed, Sep 11, 3:51 AM
gchatelet marked an inline comment as done.Wed, Sep 11, 3:53 AM
This revision was automatically updated to reflect the committed changes.