267576 - --enable-quantify configure option should imply /OPT:NOICF

Status: RESOLVED FIXED

Description

[Brian Ryner (not reading)]

The ICF optimization in the MSVC linker allows it to point identical methods at
the same address, which is a code size savings.  However, this is bad for
Quantify, since you don't know which of the methods was actually called.  So I
think we should turn this optimization off when using --enable-quantify.

Comment 1

[Brian Ryner (not reading)]

Created attachment 164500 [details] [diff] [review]
patch

Comment 2

[David Baron :dbaron: 🏴󠁵󠁳󠁣󠁡󠁿 ⌚UTC-8 (if account gets disabled due to email bounces, ask a bugzilla admin to reenable it)]

Brendan might want this off unconditionally -- I remember him complaining about
function pointer equality a while back -- I'm not sure how he solved the bug in
question.

Comment 3

[Brian Ryner (not reading)]

checked in