[PATCH] D24039: Fixed spill stack objects are mutable

[Philip Reames via llvm-commits]

reames added a comment.

In https://reviews.llvm.org/D24039#529494, @hfinkel wrote:

> In https://reviews.llvm.org/D24039#529406, @reames wrote:
>
> > At least per documentation, spill slots should be created via CreateSpillStackObject.  I think you may be fixing the wrong bug.  Even if not silently disabling all optimization for existing callers is not acceptable.
>
>
> I'm not quite sure that's right: Some ABIs have fixed-offset spill slots (e.g. PowerPC). Calling a spill slot immutable seems odd anyway, given that, by definition, you store into it. Also, they have isAliased == false regardless.


Hal, I think we have a terminology problem here.  To me a "spill slot" is one created by the register allocator when spilling vregs.  Are you saying that PowerPC has preassigned offsets which the register allocator must use?  Or are you referring to something else entirely?


Repository:
  rL LLVM

https://reviews.llvm.org/D24039