[PATCH] D39053: [Bitfield] Add more cases to making the bitfield a separate location

Hal Finkel via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 23 17:44:37 PDT 2017


hfinkel added a comment.

> With this patch we avoid unaligned loads and stores, at least on MIPS architecture.

Can you please explain the nature of the problematic situations?

Also, this patch does not update the comments that describe the splitting algorithm. That should be improved.

If this is just addressing the classic problem that, once we narrow a load we can't re-widen it, it might be best to solve this another way (e.g., by adding metadata noting the dereferenceable range).


https://reviews.llvm.org/D39053





More information about the cfe-commits mailing list