[PATCH] D24039: Fixed spill stack objects are mutable

[Philip Reames via llvm-commits]

reames added a comment.

In https://reviews.llvm.org/D24039#529495, @kparzysz wrote:

> What documentation are you referring to?


In the same header:
/// Create a new statically sized stack object that represents a spill slot,

  /// returning a nonnegative identifier to represent it.
  int CreateSpillStackObject(uint64_t Size, unsigned Alignment);

I also confirmed that the register allocator (via VirtRegMap::createSpillSlot) does use this interface.

> Second of all, the rest of your comment sounds strange given my description of the problem.  It is exactly that optimizations taking advantage of the "immutable" flag should not be taking place, given that these slots are not immutable.


My point is that while *spill slots* are clearly not immutable, there are other stack slots created by the same interface you modified which are.  Disabling optimization of those slots is clearly not okay.


Repository:
  rL LLVM

https://reviews.llvm.org/D24039