Bug 146831 - [JSC] Enable exception fuzzing for GCC too
Summary: [JSC] Enable exception fuzzing for GCC too
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Csaba Osztrogonác
URL:
Keywords:
Depends on:
Blocks: 134945
  Show dependency treegraph
 
Reported: 2015-07-10 03:35 PDT by Csaba Osztrogonác
Modified: 2015-07-22 14:33 PDT (History)
5 users (show)

See Also:


Attachments
Patch (1.58 KB, patch)
2015-07-10 03:36 PDT, Csaba Osztrogonác
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Csaba Osztrogonác 2015-07-10 03:35:24 PDT
http://trac.webkit.org/changeset/171213 added exception fuzzing,
but operationExceptionFuzz() is guarded with COMPILER(CLANG).

There is no reason to do it, it builds fine with GCC too.
Comment 1 Csaba Osztrogonác 2015-07-10 03:36:31 PDT
Created attachment 256574 [details]
Patch
Comment 2 Darin Adler 2015-07-10 09:52:19 PDT
Comment on attachment 256574 [details]
Patch

What port is building with non-clang gcc?
Comment 3 Csaba Osztrogonác 2015-07-10 09:54:42 PDT
(In reply to comment #2)
> Comment on attachment 256574 [details]
> Patch
> 
> What port is building with non-clang gcc?

EFL and GTK.
Comment 4 Csaba Osztrogonác 2015-07-22 08:43:31 PDT
ping?
Comment 5 WebKit Commit Bot 2015-07-22 14:33:16 PDT
Comment on attachment 256574 [details]
Patch

Clearing flags on attachment: 256574

Committed r187184: <http://trac.webkit.org/changeset/187184>
Comment 6 WebKit Commit Bot 2015-07-22 14:33:18 PDT
All reviewed patches have been landed.  Closing bug.