[LLVMdev] lld coding style

Renato Golin renato.golin at linaro.org
Tue Oct 7 00:47:44 PDT 2014


On 7 October 2014 08:28, Zachary Turner <zturner at google.com> wrote:
> 2) For each formatted commit F, find the corresponding unformatted commit U.
> Determine the set of all files modified between U and U+1, and reformat all
> of these files, placing them in the working copy of the new repository.
> Commit this patch to the new repository using the commit information from
> U+1.

You mean, for *every* commit in lld until now? That's not just
re-writing the history of the files, but of the project itself! There
is no guarantee that what worked in the past works with the history
re-written, what was tested then won't be tested now, people that had
patches against previous releases won't apply at all.

Even if lld had no connections whatsoever to anything else in the
external world, I'd still be against it.


> Note that variable names are outside the scope of this.

Unfortunately, that's one of the biggest changes proposed...

cheers,
--renato



More information about the llvm-dev mailing list