[PATCH] D15784: Enable 2 warnings on MSVC, turn on StringPooling & intrinsic functions

Alexander Riccio via llvm-commits llvm-commits at lists.llvm.org
Sat Jan 2 13:00:42 PST 2016

ariccio added a comment.

> Thank you for doing the analysis, this is very helpful information! From looking at the results, it appears that this is only a win for debug builds; with the other builds, the changes are negligible.


> I'm not certain that the /GF flag is worth turning on, given that this flag can increase the number of addressable sections, and we've had to work to stay under the /bigobj limit in the past. If it didn't affect /bigobj, I would have no problem turning it on.

Ahh, well, I can always dump it and leave the `/Zc:` options on, which are nice for detecting silly mistakes.


More information about the llvm-commits mailing list