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

[Alexander Riccio via llvm-commits]

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.


http://reviews.llvm.org/D15784