[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