Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Some simple improvements to MIR pretty printing #33607

Merged
merged 3 commits into from
May 15, 2016
Merged

Some simple improvements to MIR pretty printing #33607

merged 3 commits into from
May 15, 2016

Commits on May 12, 2016

  1. Some simple improvements to MIR pretty printing

    In short, this PR changes the MIR printer so that it:
    
    * places an empty line between the MIR for each item
    * does *not* write an empty line before the first BB when there are no
      var decls
    * aligns the "// Scope" comments 50 chars in (makes the output more
      readable)
    * prints the scope comments as "// scope N at ..." instead of "//
      Scope(N) at ..."
    * prints a prettier scope tree:
     * no more unbalanced delimiters!
     * no more "Parent" entry (these convey no useful information)
     * drop the "Scope()" and just print scope IDs
     * no braces when the scope is empty
    jonas-schievink committed May 12, 2016
    Configuration menu
    Copy the full SHA
    96b178b View commit details
    Browse the repository at this point in the history

Commits on May 13, 2016

  1. Configuration menu
    Copy the full SHA
    95a968c View commit details
    Browse the repository at this point in the history
  2. Indent comments less

    40 chars is still enough indentation (most common MIR statements don't
    take more than 40 chars), and fits more easily in 80-character
    terminals.
    jonas-schievink committed May 13, 2016
    Configuration menu
    Copy the full SHA
    2c7e398 View commit details
    Browse the repository at this point in the history