Details
Diff Detail
Event Timeline
include/llvm/CodeGen/GlobalISel/LegalizerHelper.h | ||
---|---|---|
135 | Comment? | |
lib/CodeGen/GlobalISel/LegalizerHelper.cpp | ||
1795 | Maybe we should assert that Size >= NarrowSize? | |
1839–1840 | Can this just be an else if? | |
1845 | Out of curiosity, could NumParts ever be 0 for some reason? If so, what would that mean? | |
1852 | Can we have a comment explaining what this lambda is doing, just for a small mental breather? :P | |
1859 | Offset/8 is used a few places here, maybe make it a variable? | |
1883 | Can we have a comment here? |
lib/CodeGen/GlobalISel/LegalizerHelper.cpp | ||
---|---|---|
1869 | The alignment isn't used here. The base alignment is copied from the MMO, and the effective alignment is computed from the base alignment + the offset. |
Comment?