Summary: | Add MeasureCacheSizes unsupported module as fallback when CPU caches can't be queried | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | Eigen | Reporter: | Benoit Jacob <jacob.benoit.1> | ||||||
Component: | Unsupported modules | Assignee: | Nobody <eigen.nobody> | ||||||
Status: | NEW --- | ||||||||
Severity: | Unknown | CC: | benoit.steiner.goog, chtz, gael.guennebaud | ||||||
Priority: | Normal | ||||||||
Version: | unspecified | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Whiteboard: | |||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 937 | ||||||||
Attachments: |
|
Description
Benoit Jacob
2015-01-16 22:34:17 UTC
Created attachment 516 [details]
Add MeasureCacheSizes unsupported module.
Comment on attachment 516 [details]
Add MeasureCacheSizes unsupported module.
Gael - please review!
Bug 931 has the patches to actually make use of this in products. See bug 937 comment 8: we mostly agree that for now we only need to know L1 and last level of cache, so only 2 values. So only the front() and back() of the vector returned by measure_cache_sizes matter. I think I'll still let this function return a vector, it's a bit nicer and if something goes wrong and it returns wrong values, the extra values in the middle of the vector can be very interesting to understand what happened. Comment on attachment 516 [details]
Add MeasureCacheSizes unsupported module.
Cancelling review because I'm burdening you a lot already and you said that for an unsupported module I dont need a review.
-- GitLab Migration Automatic Message -- This bug has been migrated to gitlab.com's GitLab instance and has been closed from further activity. You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.com/libeigen/eigen/issues/930. |