[llvm] r195067 - [Mips] Support for MicroMips STO refactoring.

Rafael EspĂ­ndola rafael.espindola at gmail.com
Mon Nov 18 22:07:17 PST 2013


> [Mips] Support for MicroMips STO refactoring.
>
> No true functional changes.
>
> Change the "hack" name of emitMipsHackSTOCG to emitSymSTO.
>
> Remove demonstration code in AsmParser for emitMipsHackSTOCG and
> emitMipsHackELFFlags. The STO field is in an ELF symbol and is not
> an explicit directive. That said, we are missing the compliment call
> in AsmParser and that will need to be addressed soon.


This is a really bad change. The existing asm support was a hack, but
this worse. Right now emitSymSTO is an object file emission only
feature. The asm streamer is a nop.

Reiterating one more time: The entire idea of having the streamer
interface is that printing assembly and reading it back produces the
same output. This is obviously not the case right now with emitSymSTO
being a nop. This is as bad as having an if on the output being an
object file or assembly.

Also, this was not reviewed and introduce warnings in the build that
Eric fixed in followup.

Please revert this. Lets make sure that in every step along the way we
are always able to print assembly and read it back in.

Cheers,
Rafael



More information about the llvm-commits mailing list