Skip to content

Commit

Permalink
delete CACHELINE
Browse files Browse the repository at this point in the history
Since #2888 this macro is no longer
used in any place.
  • Loading branch information
shyouhei committed Apr 13, 2020
1 parent c37a357 commit 5dc6080
Showing 1 changed file with 0 additions and 11 deletions.
11 changes: 0 additions & 11 deletions internal/vm.h
Expand Up @@ -23,17 +23,6 @@
# undef rb_method_basic_definition_p
#endif

/* I have several reasons to choose 64 here:
*
* - A cache line must be a power-of-two size.
* - Setting this to anything less than or equal to 32 boosts nothing.
* - I have never seen an architecture that has 128 byte L1 cache line.
* - I know Intel Core and Sparc T4 at least uses 64.
* - I know jemalloc internally has this exact same `#define CACHE_LINE 64`.
* https://github.com/jemalloc/jemalloc/blob/dev/include/jemalloc/internal/jemalloc_internal_types.h
*/
#define CACHELINE 64

struct rb_callable_method_entry_struct; /* in method.h */
struct rb_method_definition_struct; /* in method.h */
struct rb_execution_context_struct; /* in vm_core.h */
Expand Down

0 comments on commit 5dc6080

Please sign in to comment.