[PATCH] D25085: [ELF] - Do not crash if common symbol has alignment 0.

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 30 08:23:59 PDT 2016


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

LGTM. We could handle alignment 0 as alignment 1 here, but we shouldn't do that. This is a crash bug, but we've never received a bug report about it, so it is very likely that there's no such object file out there that assumes alignment 0 = alignment 1.


https://reviews.llvm.org/D25085





More information about the llvm-commits mailing list