[PATCH] D65859: [LLVM][Alignment] Fix MSVC warning
Guillaume Chatelet via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 15 04:40:50 PDT 2019
gchatelet added a comment.
In D65859#1629358 <https://reviews.llvm.org/D65859#1629358>, @RKSimon wrote:
> This isn't handling all the cases - its probably better to push at the top of the file (before namespace) and similarly pop at the end of the file
Thx Simon, I'm OOO right now and traveling next week. Would you have a chance to check that the fix you offer is fixing the Windows build?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65859/new/
https://reviews.llvm.org/D65859
More information about the llvm-commits
mailing list