This adds the majority of the AVR backend.
Details
Diff Detail
- Repository
- rL LLVM
Event Timeline
| lib/Target/AVR/AVRFrameLowering.h | ||
|---|---|---|
| 9–12 ↗ | (On Diff #49871) | Remove empty comment block |
| 30 ↗ | (On Diff #49871) | Pointless comment |
| lib/Target/AVR/AVRISelLowering.h | ||
| 119–123 ↗ | (On Diff #49871) | C++ style comments |
| lib/Target/AVR/AVRInstrInfo.cpp | ||
| 38–43 ↗ | (On Diff #49871) | These are not OK |
| lib/Target/AVR/AVRInstrInfo.h | ||
| 23 ↗ | (On Diff #49871) | Missing undef? |
| lib/Target/AVR/AVRSubtarget.h | ||
| 68 ↗ | (On Diff #49871) | this->? |
LGTM with some style nits, plus the pass setup seems to be adding a couple of passes I don't see being added
| lib/Target/AVR/AVRFrameLowering.h | ||
|---|---|---|
| 19–21 ↗ | (On Diff #53861) | C++ style comments |
| lib/Target/AVR/AVRISelLowering.h | ||
| 39 ↗ | (On Diff #53861) | Why not all caps like the others? |
| 82–83 ↗ | (On Diff #53861) | I think it's best to not duplicate the comments on overrides. The base class header comment is more likely to be kept up to date |
| lib/Target/AVR/AVRInstrInfo.cpp | ||
| 69 ↗ | (On Diff #53861) | brace on case lines |
| 390 ↗ | (On Diff #53861) | Question mark? |
| 462 ↗ | (On Diff #53861) | static_cast |
| lib/Target/AVR/AVRInstrInfo.h | ||
| 29–33 ↗ | (On Diff #53861) | C++ comments |
| lib/Target/AVR/AVRRegisterInfo.cpp | ||
| 57–69 ↗ | (On Diff #53861) | Some comments about why these registers are reserved might be helpful |
| 171 ↗ | (On Diff #53861) | Fallthrough |
| lib/Target/AVR/AVRRegisterInfo.h | ||
| 23–25 ↗ | (On Diff #53861) | C++ comments. Repeat for everywhere else |
| lib/Target/AVR/AVRSubtarget.cpp | ||
| 37 ↗ | (On Diff #53861) | Do these actually initialize to 0? I don't think I've seen empty parens like this before |
| lib/Target/AVR/AVRSubtarget.h | ||
| 38 ↗ | (On Diff #53861) | Extra lne |
| 48 ↗ | (On Diff #53861) | return on new line like the others here |
| lib/Target/AVR/AVRTargetMachine.cpp | ||
| 29 ↗ | (On Diff #53861) | This is only one place, so you can just move the literal there |
| 32 ↗ | (On Diff #53861) | static, Get->get |
| 35 ↗ | (On Diff #53861) | No return after else |
| 97 ↗ | (On Diff #53861) | Don't see this |
| 104 ↗ | (On Diff #53861) | I don't see this file? |
| lib/Target/AVR/MCTargetDesc/AVRMCTargetDesc.h | ||
| 23–25 ↗ | (On Diff #53861) | Alphabetize |
| lib/Target/AVR/AVRRegisterInfo.cpp | ||
|---|---|---|
| 57–69 ↗ | (On Diff #53861) | Agree |