dschuff accepted this revision. dschuff added a comment. This revision is now accepted and ready to land.
Looks OK to me. All of these options handle explicit overrides to the ABI which I guess just didn't work before but doesn't change any defaults. We should probably put something in the emscripten release notes with a list of the flags that now actually do something, just in case someone was accidentally using them before :) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118573/new/ https://reviews.llvm.org/D118573 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
