ICU has RTTI enabled when built with MSVC

RESOLVED INVALID

Status

()

Core
Build Config
RESOLVED INVALID
10 months ago
10 months ago

People

(Reporter: aklotz, Unassigned)

Tracking

Trunk
Unspecified
Windows
Points:
---

Firefox Tracking Flags

(firefox53 affected)

Details

(Reporter)

Description

10 months ago
I see this in warning spew during builds.

The compiler is throwing up warning D9025 because we are passing both -GR- and -GR on the same command line. The latter is overriding the former.

Comment 1

10 months ago
That's done on purpose, because ICU *does* need RTTI.
Status: NEW → RESOLVED
Last Resolved: 10 months ago
Resolution: --- → INVALID
(Reporter)

Comment 2

10 months ago
Fair enough. Any chance we could get rid of the -GR- so that it doesn't raise a false alarm?

Obviously to the untrained eye it looks like a build system bug, even thought that was intentional.
Flags: needinfo?(mh+mozilla)

Comment 3

10 months ago
Not without large changes to the python code driving moz.build.
Flags: needinfo?(mh+mozilla)
You need to log in before you can comment on or make changes to this bug.