[PATCH] D77002: Add /bigobj for SPIRV dialect on MSVC.

[Stella Laurenzo via Phabricator via llvm-commits]

stellaraccident added a comment.

In D77002#1948310 <https://reviews.llvm.org/D77002#1948310>, @mehdi_amini wrote:

> Do why know why is this object so large? This may be an indication of something underlying to look at anyway.


Afaik, it is not so much "large" as it is quirks about PE object files in debug builds with large string tables. From what I've read, VC emits a lot of sections in that case, and it is the total number of sections that overflows without /bigobj. But to answer your question, no, I did no further analysis. Most projects I work on just uncategorically set /bigobj globally for Windows/VC builds. The only reason not to do so, I am told, is compatibility with tools from prior to about 2005.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D77002/new/

https://reviews.llvm.org/D77002