On Thu, Aug 10, 2017 at 2:36 PM, Christopher Li <sparse@xxxxxxxxxxx> wrote: > I would like the manipulate of the bits representation abstract as > higher level function or macros. I have suspicious that bits > representation can be simplify in the future. Wrap them into > higher level function at the call site will be easier to read what > is the intend for this operation. > > It also make the change of the underlying bit representation easier, > that is if we do found a simpler representation. It's many months ago so I may not remember the details correctly but ... The initial version Nicolai posted (or maybe it was the second) *had* function to abstract the operations and it was, IMO, really not very readable and I asked him to change to something simpler like simply using X |= M or X &= ~M when possible. It's what he did and it was, IMO, much more readable. Meanwhile, I also tried on my side to have some nice & cleaner bit representation. It looked promising at first but was deceiving at the end. So, I think that this series is much more valuable upstreamed than waiting for some hypothetical super clean abstract layer for those few bits. Of course, if you have some concrete set of patches making this better, I would be glad to take a look at it and change my mind. Otherwise, please consider it for upstreaming directly after the release and improving the bit representation and the abstraction layer can always be done later too. -- Luc -- To unsubscribe from this list: send the line "unsubscribe linux-sparse" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
