preserve names of pattern variables in bytecode

[scabug]

(No description for SI-5739.)

[scabug]

Imported From: https://issues.scala-lang.org/browse/SI-5739?orig=1
Reporter: @xeno-by
See #5158, #6070
Attachments:

• patmat_mangled_names_of_locals.png (created on May 3, 2012 9:58:57 AM UTC, 206204 bytes)

[scabug]

@xeno-by said:
Why was the priority lowered from critical? This issue pretty much forces one to use old patmat when debugging patmat-heavy code.

[scabug]

@adriaanm said:
because I have 131 bugs assigned to me, give or take,
and I need to prioritize which ones I'll fix first
major is still pretty high on my list

[scabug]

@adriaanm said:
to further improve debuggability, we should also emit local variables for case class fields (this would then fix #5158)

[scabug]

@adriaanm said:
scala/scala#937

[scabug]

@adriaanm said:
reopening because it causes performance regressions

[scabug]

@adriaanm said:
scala/scala#1060

[scabug]

@adriaanm said:
reopening for a further tweak: shouldn't emit a local variable for debuggability if the original name was a wildcard -- it seems unlikely to matter and impacts bytecode size unnecessarily

[scabug]

@adriaanm said (edited on Feb 3, 2013 5:49:53 PM UTC):
refined in scala/scala#2033