Changeset 83860 in webkit


Ignore:
Timestamp:
Apr 14, 2011 8:47:38 AM (13 years ago)
Author:
satish@chromium.org
Message:

2011-04-14 Satish Sampath <satish@chromium.org>

Reviewed by Anders Carlsson.

Don't emit RegExp tables for chromium where they are not used
https://bugs.webkit.org/show_bug.cgi?id=58544

  • JavaScriptCore.gyp/JavaScriptCore.gyp:
  • create_regex_tables: Added the "--notables" command line argument.
Location:
trunk/Source/JavaScriptCore
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/JavaScriptCore/ChangeLog

    r83827 r83860  
     12011-04-14  Satish Sampath  <satish@chromium.org>
     2
     3        Reviewed by Anders Carlsson.
     4
     5        Don't emit RegExp tables for chromium where they are not used
     6        https://bugs.webkit.org/show_bug.cgi?id=58544
     7
     8        * JavaScriptCore.gyp/JavaScriptCore.gyp:
     9        * create_regex_tables: Added the "--notables" command line argument.
     10
    1112011-04-14  Patrick Gansterer  <paroga@webkit.org>
    212
  • trunk/Source/JavaScriptCore/JavaScriptCore.gyp/JavaScriptCore.gyp

    r80389 r83860  
    207207            '../create_regex_tables',
    208208          ],
     209          'arguments': [
     210            '--no-tables',
     211          ],
    209212          'outputs': [
    210213            '<(INTERMEDIATE_DIR)/RegExpJitTables.h',
    211214          ],
    212           'action': ['python', '<@(_inputs)', '<@(_outputs)'],
     215          'action': ['python', '<@(_inputs)', '<@(_arguments)', '<@(_outputs)'],
    213216        },
    214217      ],
  • trunk/Source/JavaScriptCore/create_regex_tables

    r78042 r83860  
    3636arrays = "";
    3737functions = "";
     38emitTables = (len(sys.argv) < 2 or sys.argv[1] != "--no-tables")
    3839
    3940for name, classes in types.items():
     
    5758    ranges.sort();
    5859   
    59     if classes["UseTable"] and (not "Inverse" in classes):
     60    if emitTables and classes["UseTable"] and (not "Inverse" in classes):
    6061        array = ("static const char _%sData[65536] = {\n" % name);
    6162        i = 0
     
    8889    function += ("CharacterClass* %sCreate()\n" % name)
    8990    function += ("{\n")
    90     if classes["UseTable"]:
     91    if emitTables and classes["UseTable"]:
    9192        if "Inverse" in classes:
    9293            function += ("    CharacterClass* characterClass = new CharacterClass(CharacterClassTable::create(_%sData, true));\n" % (classes["Inverse"]))
Note: See TracChangeset for help on using the changeset viewer.