[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