Remove clang-cl command-line argument workarounds

RESOLVED FIXED in Firefox 47

Status

defect
RESOLVED FIXED
4 years ago
Last year

People

(Reporter: Ehsan, Unassigned)

Tracking

unspecified
mozilla47
Dependency tree / graph

Firefox Tracking Flags

(firefox47 fixed)

Details

Attachments

(1 attachment)

Reporter

Description

4 years ago
In https://llvm.org/bugs/show_bug.cgi?id=26090, we changed the default
behavior of clang-cl to accept unknown command line arguments.  This
patch removes the Firefox workaround for this issue.
Reporter

Comment 1

4 years ago
Attachment #8711858 - Flags: review?(mh+mozilla)
Reporter

Updated

4 years ago
Blocks: winclang
Comment on attachment 8711858 [details] [diff] [review]
Remove clang-cl command-line argument workarounds

Review of attachment 8711858 [details] [diff] [review]:
-----------------------------------------------------------------

Maybe we'd want to add a version test on clang-cl?
Attachment #8711858 - Flags: review?(mh+mozilla) → review+
Reporter

Comment 3

4 years ago
(In reply to Mike Hommey [:glandium] from comment #2)
> Maybe we'd want to add a version test on clang-cl?

We still don't support anything older than the tip of tree.  :-)  Various things will be broken if someone tries anything older.

I'm hoping that maybe 3.9 will end up being the first clang-cl release that we fully support, but 3.8 was just branched a week ago or so...
Reporter

Comment 4

4 years ago
CCing people who build with clang.  You want to be at ToT!

Comment 6

4 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/08f0995ef873
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
Target Milestone: mozilla46 → mozilla47

Updated

Last year
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.