[PATCH] D16154: Default vaarg lowering should support indirect struct types.

Derek Schuff via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 17 10:59:55 PST 2016


dschuff added a comment.

This LGTM for PNaCl and WebAssembly.

It's possible that for wasm we may still end up doing our own thing for varargs instead of using the default, but this still looks like strictly an improvement, (and it makes bootstrapping a backend simpler).


http://reviews.llvm.org/D16154





More information about the cfe-commits mailing list