This wraps up from D119053. The 2 headers are moved as described,
fixed file headers and include guards, updated all files where the old
paths were detected (simple grep through the repo), and clang-format-ed it all.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
llvm/include/llvm/CodeGen/RegisterBankInfo.h | ||
---|---|---|
220 | ah - clang-format handled the moved file as a new file; but I'd say let's let it make these changes, they are goodness anyway (and small) |
Comment Actions
As this is a follow-up of a refactoring, I assume I can just land it (short of file header comments, there was nothing really this patch did more intelligently)
llvm/lib/Target/ARM/ARMTargetMachine.cpp | ||
---|---|---|
43 | Hmm...did you use clang-format-diff.py? I wonder why this was changed (although it's a legit one) |
llvm/lib/Target/ARM/ARMTargetMachine.cpp | ||
---|---|---|
43 | did git clang-format, and since that had to re-sort the includes due to my change, it affected incorrectly placed lines. |
ah - clang-format handled the moved file as a new file; but I'd say let's let it make these changes, they are goodness anyway (and small)