• Peter Marshall's avatar
    [cleanup] Don't declare inline runtime functions by default · 49420760
    Peter Marshall authored
    For each intrinsic/runtime function we define in runtime.h, an inline
    version is automatically declared. We only ever use 24 of the inline
    functions. Even though we don't call the other ones, macro magic means
    they still take up space by existing in various arrays and tables like
    kIntrinsicFunctions. They also create code in switch statements.
    
    Some drive-by cleanups:
     - Remove the switch in NameForRuntimeId() and just use the table of
       runtime functions to lookup the name directly.
     - Remove tests for IsFunction, ClassOf and StringAdd intrinsics as
       they are the last users of the inline versions of these.
     - Remove the MaxSmi inline version as it is only used in tests.
    
    Saves 64 KiB binary size.
    
    Change-Id: I4c870ddacd2655ffcffa97d93200ed8f853752f5
    Reviewed-on: https://chromium-review.googlesource.com/c/1261939
    Commit-Queue: Peter Marshall <petermarshall@chromium.org>
    Reviewed-by: 's avatarBenedikt Meurer <bmeurer@chromium.org>
    Reviewed-by: 's avatarLeszek Swirski <leszeks@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#56412}
    49420760
test-run-intrinsics.cc 1.61 KB