[PATCH] D36379: LLVM: Make GNU COFF Aligncomm optional

Martell Malone via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 7 07:41:29 PDT 2017


martell added a comment.

In https://reviews.llvm.org/D36379#833526, @majnemer wrote:

> GCC already has such a flag, it calls it `-mpe-aligned-commons` and it is on by default. I don't think we should stop emitting aligncomm for GNU targets by default, we shouldn't (and don't AFAIK) emit aligncomm for MSVC targets.


Thanks @majnemer, in that case we should also have a flag `-mpe-aligned-commons` and be able to call `-mno-pe-aligned-commons` to disable it for mingw at compile time if desired.
The question I have is generally about how we propagate this flag all the way down into llvm in `MCWinCOFFStreamer.cpp`


Repository:
  rL LLVM

https://reviews.llvm.org/D36379





More information about the llvm-commits mailing list