This is an archive of the discontinued LLVM Phabricator instance.

Differential D119519

[NFC] Sort textual headers by alphabetical order
ClosedPublic

Authored by SixWeining on Feb 10 2022, 9:59 PM.

Details

Reviewers
rastogishubham
rengolin
Commits
rG0e263f17751d: [NFC] Sort textual headers by alphabetical order

Diff Detail

Unit TestsFailed

TimeTest
60,110 msx64 debian > Clang.CodeGen/RISCV/rvv-intrinsics::vloxseg.c
60,190 msx64 debian > Clang.CodeGen/RISCV/rvv-intrinsics::vluxseg.c
60,120 msx64 debian > Clang.CodeGen/RISCV/rvv-intrinsics-overloaded::vloxseg.c
60,170 msx64 debian > Clang.CodeGen/RISCV/rvv-intrinsics-overloaded::vluxseg.c
60,040 msx64 debian > MLIR.Examples/standalone::test.toy

Event Timeline

SixWeining requested review of this revision.Feb 10 2022, 9:59 PM
SixWeining created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptFeb 10 2022, 10:00 PM
Herald added a subscriber: llvm-commits. · View Herald Transcript
rastogishubham accepted this revision.Feb 10 2022, 10:00 PM
This revision is now accepted and ready to land.Feb 10 2022, 10:00 PM
SixWeining mentioned this in D115859: [LoongArch 2/6] Add ELF machine flag and relocs for upcoming LoongArch target.Feb 10 2022, 10:02 PM
SixWeining added a comment.Feb 10 2022, 10:09 PM

Hi @rastogishubham, thanks for your review. I don't have commit access yet, could you help to land this patch?

Harbormaster completed remote builds in B148914: Diff 407772.Feb 10 2022, 10:28 PM
This revision was landed with ongoing or failed builds.Feb 10 2022, 10:35 PM
Closed by commit rG0e263f17751d: [NFC] Sort textual headers by alphabetical order (authored by Shubham Rastogi <shubhamrastogi@Shubhams-MacBook-Pro.local>). · Explain Why
This revision was automatically updated to reflect the committed changes.
Shubham Rastogi <shubhamrastogi@Shubhams-MacBook-Pro.local> added a commit: rG0e263f17751d: [NFC] Sort textual headers by alphabetical order.
rastogishubham added a comment.Feb 10 2022, 10:35 PM

Done :)

SixWeining added a comment.Feb 10 2022, 10:41 PM
In D119519#3313470, @rastogishubham wrote:

Done :)

Thank you!

Revision Contents

PathSize
llvm/
include/
llvm/
2 lines

Diff 407772

llvm/include/llvm/module.modulemap

Show First 20 Lines • Show All 65 Lines • ▼ Show 20 Linesmodule LLVM_BinaryFormat {
textual header "BinaryFormat/ELFRelocs/ARM.def" textual header "BinaryFormat/ELFRelocs/ARM.def"
textual header "BinaryFormat/ELFRelocs/ARC.def" textual header "BinaryFormat/ELFRelocs/ARC.def"
textual header "BinaryFormat/ELFRelocs/AVR.def" textual header "BinaryFormat/ELFRelocs/AVR.def"
textual header "BinaryFormat/ELFRelocs/BPF.def" textual header "BinaryFormat/ELFRelocs/BPF.def"
textual header "BinaryFormat/ELFRelocs/CSKY.def" textual header "BinaryFormat/ELFRelocs/CSKY.def"
textual header "BinaryFormat/ELFRelocs/Hexagon.def" textual header "BinaryFormat/ELFRelocs/Hexagon.def"
textual header "BinaryFormat/ELFRelocs/i386.def" textual header "BinaryFormat/ELFRelocs/i386.def"
textual header "BinaryFormat/ELFRelocs/Lanai.def" textual header "BinaryFormat/ELFRelocs/Lanai.def"
textual header "BinaryFormat/ELFRelocs/LoongArch.def"
textual header "BinaryFormat/ELFRelocs/M68k.def" textual header "BinaryFormat/ELFRelocs/M68k.def"
textual header "BinaryFormat/ELFRelocs/Mips.def" textual header "BinaryFormat/ELFRelocs/Mips.def"
textual header "BinaryFormat/ELFRelocs/MSP430.def" textual header "BinaryFormat/ELFRelocs/MSP430.def"
textual header "BinaryFormat/ELFRelocs/PowerPC64.def" textual header "BinaryFormat/ELFRelocs/PowerPC64.def"
textual header "BinaryFormat/ELFRelocs/PowerPC.def" textual header "BinaryFormat/ELFRelocs/PowerPC.def"
textual header "BinaryFormat/ELFRelocs/RISCV.def" textual header "BinaryFormat/ELFRelocs/RISCV.def"
textual header "BinaryFormat/ELFRelocs/Sparc.def" textual header "BinaryFormat/ELFRelocs/Sparc.def"
textual header "BinaryFormat/ELFRelocs/SystemZ.def" textual header "BinaryFormat/ELFRelocs/SystemZ.def"
textual header "BinaryFormat/ELFRelocs/VE.def" textual header "BinaryFormat/ELFRelocs/VE.def"
textual header "BinaryFormat/ELFRelocs/x86_64.def" textual header "BinaryFormat/ELFRelocs/x86_64.def"
textual header "BinaryFormat/WasmRelocs.def" textual header "BinaryFormat/WasmRelocs.def"
textual header "BinaryFormat/MsgPack.def" textual header "BinaryFormat/MsgPack.def"
textual header "BinaryFormat/ELFRelocs/LoongArch.def"
} }
module LLVM_Config { module LLVM_Config {
requires cplusplus requires cplusplus
umbrella "Config" umbrella "Config"
extern module LLVM_Extern_Config_Def "module.extern.modulemap" extern module LLVM_Extern_Config_Def "module.extern.modulemap"
module * { export * } module * { export * }
} }
▲ Show 20 Lines • Show All 344 Lines • Show Last 20 Lines