This is an archive of the discontinued LLVM Phabricator instance.

[libcxx] [test] Portability fix: add missing header for std::max_align_t.
ClosedPublic

Authored by amakc11 on Nov 16 2018, 12:52 PM.

Details

Reviewers
EricWF
ldionne
Summary

Some tests use type std::max_align_t, but don't include <cstddef> header directly. As a result, these tests won't compile against some conformant libraries.

Diff Detail

Repository
rCXX libc++

Event Timeline

amakc11 created this revision.Nov 16 2018, 12:52 PM
ldionne accepted this revision.Nov 19 2018, 8:47 AM

Thanks! Do you have many other patches like this one? If so, could you consider lumping them together so I don't have to commit many of them?

This revision is now accepted and ready to land.Nov 19 2018, 8:47 AM
ldionne closed this revision.Nov 19 2018, 9:44 AM

Committed as r347232. Thanks.