3 commits

Author	SHA1	Message	Date
Elliott Hughes	cb239bda65	Simplify atoi*/strto* for signed integers. ... Make the cost of strto<signed> closer to the cost of strto<unsigned> by removing an `if` from the inner loop. Previously a signed conversion cost 10ns more than an unsigned one. After: BM_inttypes_strtoimax 81 ns 81 ns 8603362 BM_inttypes_strtoumax 78 ns 78 ns 8967174 BM_stdlib_strtol 81 ns 81 ns 8685537 BM_stdlib_strtoll 81 ns 81 ns 8685481 BM_stdlib_strtoul 78 ns 78 ns 8962569 BM_stdlib_strtoull 78 ns 78 ns 8972023 Bug: N/A Test: ran tests, benchmarks Change-Id: I72dd5499427b6a940bd94c4d6f727f7efe134d7e	2017-12-21 17:44:22 -08:00
Elliott Hughes	1133fecde3	Use inline ctype functions for ato*/strto* and scanf. ... This also let us remove the `unsigned char` casts, since we define our inlines to work for all values. Before: Benchmark Time CPU Iterations ------------------------------------------------------------- BM_inttypes_strtoimax 112 ns 112 ns 6222193 BM_inttypes_strtoumax 104 ns 104 ns 6725010 BM_stdlib_strtol 113 ns 113 ns 6195861 BM_stdlib_strtoll 113 ns 113 ns 6195633 BM_stdlib_strtoul 105 ns 105 ns 6691394 BM_stdlib_strtoull 105 ns 105 ns 6690695 BM_stdio_scanf_d 504 ns 503 ns 1385224 BM_stdio_scanf_maps 1900 ns 1898 ns 369260 BM_stdio_scanf_maps_baseline 1030 ns 1030 ns 678832 BM_stdio_scanf_s 433 ns 432 ns 1619086 After: BM_inttypes_strtoimax 91 ns 91 ns 7718194 BM_inttypes_strtoumax 82 ns 82 ns 8508052 BM_stdlib_strtol 92 ns 92 ns 7674694 BM_stdlib_strtoll 91 ns 91 ns 7639228 BM_stdlib_strtoul 83 ns 82 ns 8500304 BM_stdlib_strtoull 82 ns 82 ns 8504929 BM_stdio_scanf_d 465 ns 465 ns 1507891 BM_stdio_scanf_maps 1836 ns 1836 ns 381082 BM_stdio_scanf_maps_baseline 846 ns 845 ns 830881 BM_stdio_scanf_s 419 ns 419 ns 1671979 Bug: N/A Test: ran tests, benchmarks Change-Id: I44681daf16c4328b060770cf11fc0633157c427f	2017-12-19 16:37:01 -08:00
Elliott Hughes	1921dce886	Refactor the ato* and strto* family. ... There are no meaningful changes here, just a minimal conversion to two C++ templates to make further changes easier. Bug: N/A Test: ran tests, benchmarks Change-Id: I958fbf17a85f19dd8f17bfb4bbb9314d220daa3b	2017-12-19 13:55:54 -08:00