Any objections to running clang-format in include/llvm/Support/ELF.h?
Davide Italiano via llvm-commits
llvm-commits at lists.llvm.org
Mon Nov 14 12:45:45 PST 2016
On Mon, Nov 14, 2016 at 12:34 PM, Rafael EspĂndola via llvm-commits
<llvm-commits at lists.llvm.org> wrote:
> The main difference will be that some enum definitions will go from
>
> long_val = 42, // comment
> val = 42, // comment
>
> to
>
> long_val = 42, // comment
> val = 43, // comment
>
> I like the aligned =, but it is somewhat annoying to have one file
> with a different style.
>
> Cheers,
> Rafael
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
No objections (actually in favour of).
I have a large'ish patch converting from ErrorOr<> to Expected<>, so
reformatting will cause a bit of merge pain to me.
If you want, I can take care of that once I'm done with the patch?
Otherwise I'll bite the bullet and resolve the conflicts.
--
Davide
--
Davide
"There are no solved problems; there are only problems that are more
or less solved" -- Henri Poincare
More information about the llvm-commits
mailing list