[PATCH] [lld][PECOFF] Fix section alignment bug in ReaderCOFF
Rui Ueyama
ruiu at google.com
Fri Sep 6 11:50:44 PDT 2013
LGTM. Thank you for finding this bug.
This patch virtually rolled back my changes to the tests, resurrecting the original CHECK conditions. That means that the test coverage is not sufficient. We should write a new test the alignment restriction, but anyway, this patch is LGTM.
Do you want me to check this in?
http://llvm-reviews.chandlerc.com/D1620
More information about the llvm-commits
mailing list