Ticket #33 (closed RFC: fixed)

Opened 6 years ago

Last modified 6 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 6 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 6 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 6 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 6 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.