[llvm] [NFC][LLVM] Fix Intrinsics.td to adhere to 80 col limit (PR #199346)

Oliver Hunt via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 1 16:51:30 PDT 2026


ojhunt wrote:

> Regarding git blame, it's already affected, so reverting won't fix this. But we should not do it again.
> 
> > _If_ you think we should be reformatting all incorrectly formatted files you should start with an RFC.
> 
> We should not. I added a comment on the RFC.

Oh absolutely I agree - to me it's incredibly bad to blindly reformat files for no other reason than to "conform to the current coding style" (regardless of type), however @jurahul is clearly of the opinion that there is value in doing so. If that is true there should be a discussion that covers all of the pros and cons of doing so and we should make a concrete decision to do so as a community rather than random ad hoc cases like this one.

https://github.com/llvm/llvm-project/pull/199346


More information about the llvm-commits mailing list