[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