On 09/16/2016 04:10 AM, Bernd Schmidt wrote:
On 09/16/2016 12:10 PM, Trevor Saunders wrote:
ok, going through all the casts added here I see the following reasons
for them.

- in md files operands is a array of rtx, we should probably have a
  different way to pass insns to the C code here.  That seems worth
  investigating incrementally.

- JUMP_LABEL can be a return which is not an insn.  That also seems
  like something that should be improved at some point.

These just show that fundamentally, rtl is just dynamically typed, and
used as such, which is why I was never massively enthusiastic about the
rtx -> rtx_insn conversion to begin with.
RTL is dynamically typed, but I think we can start carving off things like the insn chain objects -- they don't need to be dynamically typed. Essentially in my mind the insn chain doesn't need to be RTL. It's really an RTL container.

It does mean there are things we have to fix up, but so far the things we've fixed up, I've considered clear improvements.


Reply via email to