tmatheson accepted this revision.
tmatheson added a comment.
This revision now requires review to proceed.

In D146242#4428051 <https://reviews.llvm.org/D146242#4428051>, @chill wrote:

> But there isn't any other power of two between 8 and 16.

Ok, I see where I was going wrong, misreading C4 (it's the stack address which 
is "next multiple of N", which applies an alignment of N).

In that case I don't have any more objections.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D146242/new/

https://reviews.llvm.org/D146242

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to