[PATCH] D113809: [MachO] Fix struct size assertion

Wouter van Oortmerssen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 22 11:29:15 PST 2021


aardappel added a comment.

I tried, but this will take me longer to set up, which I can't do right now. I may be able to return to this later if I am still blocked by it.

I would highly recommend simply not using hardcoded constants like that in portable code however, or to clearly #ifdef whatever platform you are assuming this should be valid for.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113809



More information about the llvm-commits mailing list