vitalybuka wrote:

> > > Quick question for my understanding: With `spr` we don't get meaningful 
> > > commit messages anymore? That's quite unfortunate...
> > 
> > 
> > Can you please clarify?
> 
> The commit messages in this PR are `initial version` and `Update const.cpp`, 
> which is totally useless if I came across this in `git log`. I meanwhile 
> remembered that the policy in LLVM is to squash-and-merge, so I guess you can 
> supply a more useful message at that point, but I would still argue that the 
> history deserves better than what is currently in this PR.

LLVM repo does not allow "merge" or "rebase", everything is squash, with ONLY 
commit message from the description. 
With or without spr they they are irrelevant. 

https://github.com/llvm/llvm-project/pull/102859
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to