[PATCH] D71504: [PowerPC] Enable sret arguments.

Hubert Tong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 17 07:18:36 PST 2019


hubert.reinterpretcast added a comment.

In D71504#1787758 <https://reviews.llvm.org/D71504#1787758>, @ZarkoCA wrote:

> I don't know what happens in the case where the front-end doesn't handle some cases correctly but the back-end does.  Is there a danger in enabling this in the back-end knowing we don't get correct IR for complex types for AIX?  Is there an error in the front-end for those before we get to this step?


There's no error from the front-end. In terms of continuous delivery, this leads to silently bad code that could lead to incorrect output or crashes at runtime.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71504





More information about the llvm-commits mailing list