ro added a comment.

In D87615#2271646 <https://reviews.llvm.org/D87615#2271646>, @efriedma wrote:

> 



> X86Subtarget::initSubtargetFeatures claims that that stack is 16-byte aligned 
> on Solaris; if that's wrong, we should fix it, not force stack realignment.

I missed that (my usual issue of config info spread over llvm and clang).  I'll 
shortly post an alternative patch using that approach.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87615

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

Reply via email to