[PATCH] D10937: [yaml2obj] Align section content using AddressAlign field's value

Sean Silva chisophugis at gmail.com
Mon Jul 6 16:57:14 PDT 2015


Nice catch. The test seems really brittle though, since it depends on the exact offset. Can you instead ask for an alignment larger than the previous default, then regex check that the offset has sufficient trailing zeros? (hopefully make the alignment large enough that there is a low probability that it could happen by accident, but small enough that the resulting filesize isn't enormous/the test is still fast to execute)


Repository:
  rL LLVM

http://reviews.llvm.org/D10937







More information about the llvm-commits mailing list