Ticket #33 (closed RFC: fixed)

Opened 13 years ago

Last modified 13 years ago

[proposal] remove compilers/bcg

Reported by: kjs Owned by: whiteknight
Priority: minor Milestone: 0.9.0
Component: none Version:
Severity: low Keywords: bcg
Cc: parrot-dev@… Language:
Patch status: Platform:

Description

compilers/bcg is not used by any other component in parrot. For purposes of bytecode generation from PIR, its design is suboptimal (it won't be used by PIRC). I suggest to remove compilers/bcg, as pretty much all of the code in there is not involved with actual bytecode generation details (which would be the interesting part).

kjs

Change History

Changed 13 years ago by whiteknight

  • owner set to whiteknight
  • milestone set to 0.8.2

a few more +1s, and I'll definitely pull this out for the release.

--Andrew whitworth

Changed 13 years ago by whiteknight

  • milestone changed from 0.8.2 to 0.9.0

I'm moving this to the 0.9.0 milestone instead. We haven't gotten any feedback on this idea, and I don't want to rip out any major components this close to the tuesday release.

After Tuesday, we can look more closely at ripping out compilers/bcg/, if it's truely not being used.

--Andrew Whitworth

Changed 13 years ago by whiteknight

I'm ripping this code out right now, but it appears to me that a few references are made to it in compilers/pirc/src. All the new PIRC work is being done in compilers/pirc/new/, right? If so, then this shouldn't be a problem and I'll continue with the surgery. Good?

--Andrew Whitworth

Changed 13 years ago by whiteknight

  • status changed from new to closed
  • resolution set to fixed

Actually, on second-look it doesn't appear this is used anywhere. Deleted in r34024

--Andrew Whitworth

Note: See TracTickets for help on using tickets.