This is an archive of the discontinued LLVM Phabricator instance.

[X86] Fix stack alignment for MCU target (Clang part)
ClosedPublic

Authored by anadolskiy on Dec 18 2015, 10:02 AM.

Diff Detail

Event Timeline

anadolskiy retitled this revision from to [X86] Fix stack alignment for MCU target (Clang part).
anadolskiy updated this object.
anadolskiy added a reviewer: rjmccall.
anadolskiy added a subscriber: cfe-commits.
rjmccall added inline comments.Dec 27 2015, 11:05 PM
tools/clang/lib/AST/ASTContext.cpp
1902

Please turn this into a method or flag on TargetInfo. I suggest

bool AllowLargerPreferredTypeAlignment;
anadolskiy updated this revision to Diff 43788.Dec 30 2015, 5:52 AM

Thanks for review John. Addressed your comment.

rjmccall edited edge metadata.Jan 3 2016, 8:50 AM

LGTM, thanks.

This revision was automatically updated to reflect the committed changes.