[PATCH] D30416: [BitfieldShrinking] Shrink Bitfields load/store when the bitfields are legal to access independently

[David Li via Phabricator via llvm-commits]

davidxl added a comment.

Looks like this thread has gone stale for a while.

I have not read the patch in details so what I said might be be nonsense :)  From reading of the discussions, it seems like that using DemandedBits Analysis is not ready today to handle forward walking for load shrinking, nor ideal for backward walking without using capping,  so what is the good path going forward?   Is it possible to keep the core of this patch but with more re-use of DemandedBit analysis (with refactoring)?   If not, we may want to consider moving on with a stop-gap solution for now and committing on longer term unification as follow ups.


Repository:
  rL LLVM

https://reviews.llvm.org/D30416