Performance optimization

victor-homyakov commented 5 years ago

Changes proposed in this pull request

benchmarking code and documentation are updated
utils.isSimple is optimized
Set is removed because more thorough benchmarks showed no performance improvement in Node.js

Checklist

[x] Documentation changed
[ ] Tests added
[x] Benchmark checked

Benchmark result

Compare 8.9.4 (rev1) with latest version in PR (rev2):

Percentile:  0.5
{ rev1: 4.963768,
  rev2: 4.771325,
  'diff abs': 0.19244299999999992,
  'diff percent': 3.8769539591697266 }

Percentile:  0.9
{ rev1: 7.333798,
  rev2: 7.017333,
  'diff abs': 0.316465,
  'diff percent': 4.315158394054485 }

Percentile:  0.95
{ rev1: 8.290766,
  rev2: 7.943027,
  'diff abs': 0.3477389999999998,
  'diff percent': 4.19429278307939 }

8 9 4-vs-latest-repeat-10000

coveralls commented 5 years ago

Coverage decreased (-0.004%) to 99.403% when pulling 4a9b42199f480fab1a097f404f6d48a1a633d6a2 on victor-homyakov:performance-optimization into 844b87f666e84831b00f8da359d0194b5d5ffca1 on bem:master.

coveralls commented 5 years ago

Coverage decreased (-0.004%) to 99.403% when pulling 4a9b42199f480fab1a097f404f6d48a1a633d6a2 on victor-homyakov:performance-optimization into 844b87f666e84831b00f8da359d0194b5d5ffca1 on bem:master.

coveralls commented 5 years ago

Coverage decreased (-0.004%) to 99.403% when pulling 4a9b42199f480fab1a097f404f6d48a1a633d6a2 on victor-homyakov:performance-optimization into 844b87f666e84831b00f8da359d0194b5d5ffca1 on bem:master.

bem / bem-xjst