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 |