[PATCH] D14382: [ELF2] - Basic implementation of -r/--relocatable

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 22 22:23:29 PST 2016


>I just tried applying this patch. I noticed two things:
>
>* It is not git-clang-format clean. In particular some lines are more
>than 80 chars long.
>* It doesn't compile on linux:
>
>/home/espindola/llvm/llvm/tools/lld/ELF/OutputSections.h:306:53:
>error: unknown type name 'uintX_t'; did you mean 'uint8_t'?
> StaticRelocSection(StringRef Name, uint32_t Type, uintX_t Flags, bool IsRela);
>                                                    ^~~~~~~
>                                                    uint8_t

Sorry about that. I`ll check. Btw that reminded me about other already commited files I saw in lld.
Recently I tried to apply clang-format to sources and found that there are few places where
code is not formatted. About 50% of files are affected I think.
Can I format them all and commit as single separate patch ?

George.



More information about the llvm-commits mailing list