[PATCH] D104808: [clang][emscripten] Reduce alignof long double from 16 to 8 bytes

Alon Zakai via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 2 10:46:36 PDT 2021


kripken accepted this revision.
kripken added a comment.
This revision is now accepted and ready to land.

I believe we have all agreed that this is the right path forward, and so this can land?

Doing this change now in emscripten does not prevent us from changing the emscripten ABI again later, as the ABI is unstable (which is why we can make this change itself). We definitely should in the future consider unifying the emscripten triple with WASI, but for now, this change will be a strict improvement in emscripten.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104808



More information about the cfe-commits mailing list