[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 02:39:39 PDT 2026


ojhunt wrote:

What was the purpose or value of this - ignoring downstream conflicts which are annoying - this breaks bit blame, etc.

There are endless .td files, for which 80 column has not be enforced, are we proposing changing all of them without any actual reason? Is this the start of a general "[NFC] bring all files into accordance with the current coding standards"?

> .td files are source code. Since there is no linting for this that happens, its likely there are escapes

Diagnostics.td disagree with this arbitrary seeming decision. I've had PRs rejected previously due to code formatting changes bringing otherwise unchanged code into "conformance", so blindly reformatting an entire file for no other purpose seem incredibly bizarre.

_If_ you think we should be reformatting all incorrectly formatted files you should start with an RFC.

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


More information about the llvm-commits mailing list