[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