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

Icode file name should include phase name or id #7773

Closed
scabug opened this issue Aug 21, 2013 · 4 comments
Closed

Icode file name should include phase name or id #7773

scabug opened this issue Aug 21, 2013 · 4 comments
Assignees
Milestone

Comments

@scabug
Copy link

scabug commented Aug 21, 2013

Since -Xprint-icode is a PhasesSetting, it should be prepared to emit files for multiple phases, such as -Xprint-icode:icode,dce.

To distinguish the output files, it should encode either the phase name or the phase id in the file name.

@scabug
Copy link
Author

scabug commented Aug 21, 2013

Imported From: https://issues.scala-lang.org/browse/SI-7773?orig=1
Reporter: @som-snytt
Affected Versions: 2.11.0-M4

@scabug
Copy link
Author

scabug commented Aug 22, 2013

@magarciaEPFL said:
The functionality you're looking for is already provided by scala/scala@cb19813

In detail, -Xprint-icode isn't the only way to obtain .icode files, that commit allows -Xprint: to emit .icode files right after the chosen icode-phase. Very useful when debugging the old optimizer.

@scabug
Copy link
Author

scabug commented Feb 26, 2014

@gkossakowski said:
Related: scala/scala#3586

@scabug
Copy link
Author

scabug commented Jun 7, 2015

@som-snytt said:
scala/scala#4543

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants