[cfe-commits] [PATCH]Clang/PNaCl: add checks for alignment of basic datatypes. Fix nits.

Ivan Krasin krasin at chromium.org
Mon Aug 29 14:05:39 PDT 2011


Hi Clang team!

This CL improves test coverage for PNaClTargetInfo. I have found a way
(well, stolen it from other part of nacl codebase) to test the
alignments with FileCheck, so adding checks for the alignment of basic
datatypes.
Also it fixes two nits: wrong alignment for double (it was 4, but 8 is
desired) and added checks for _REENTRANT define.

The patch is attached and is also available online:
http://codereview.chromium.org/7794004/

Please, let me know if it's fine to commit.

-- krasin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pnacl_align_test.diff
Type: text/x-patch
Size: 2534 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110829/6fd260e1/attachment.bin>


More information about the cfe-commits mailing list