LorenzoMauro wrote:

Thank you @HazardyKnusperkeks for the review.

@owenca I agree — I also don't see why anyone would want the current behavior, 
so it definitely seems like a bug to me as well.
It's absolutely possible to fix this without introducing a new option: the 
check I wrote could simply apply unconditionally when ``BinPackParameters`` is 
set to ``AlwaysOnePerLine``.

If we take that approach, most of the changes in the PR would also become 
unnecessary.

Let me know how you'd like to proceed, and I’ll update the patch accordingly.

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

Reply via email to