This adds AVRISelLowering.cpp
Details
Diff Detail
- Repository
- rL LLVM
Event Timeline
llvm/trunk/lib/Target/AVR/AVRISelLowering.cpp | ||
---|---|---|
98–99 | Setting these to Expand should be unnecessary since you don't add i32/i64 as legal types (same for all of the other operations) | |
101 | Capitalize | |
845–847 | Don't repeat the documentation | |
850 | Comment doesn't make sense to me here | |
1135 | nullptr | |
1137–1141 | Why not just immediately cast to Function and then remove GV variable | |
1322–1327 | Formatting | |
1510 | static, no inline | |
1675 | !CallOperandVal | |
1783–1784 | You should be able to avoid a cast with getSubtarget<AVRSubtarget> |
Code review from Matt
- Don't both setting operations to expand on already illegal types
- Clean up some comments
- Use nullptr rather than 0
Setting these to Expand should be unnecessary since you don't add i32/i64 as legal types (same for all of the other operations)