Ticket #1885 (new RFC)
Parrot_load_bytecode should only load bytecode
Reported by: | whiteknight | Owned by: | cotto |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | core | Version: | master |
Severity: | medium | Keywords: | pir, pbc |
Cc: | Language: | ||
Patch status: | Platform: | all |
Description
The function src/packfile.c:Parrot_load_bytecode() is too magical. It not only loads bytecode, but it also does some rudimentary extension string matching to also allow it to load .pir and .pasm file. This is extremely magical and is not made clear through the function's name.
This function is exposed primarily through the load_bytecode PIR op, which has similar magical behavior.
I suggest that we eliminate the magical behavior of this function and the opcode that exposes it: load_bytecode and Parrot_load_bytecode should *only* load bytecode files under any extension. If the user wants to compile a PIR source file they should use the PIR compreg.