<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>Issue</th>
<td>
<a href=https://github.com/llvm/llvm-project/issues/154760>154760</a>
</td>
</tr>
<tr>
<th>Summary</th>
<td>
Clang-tidy invalid suggestion: misc-use-internal-linkage
</td>
</tr>
<tr>
<th>Labels</th>
<td>
clang-tidy
</td>
</tr>
<tr>
<th>Assignees</th>
<td>
</td>
</tr>
<tr>
<th>Reporter</th>
<td>
joker-eph
</td>
</tr>
</table>
<pre>
With clang-tidy built at current head and applying it on MLIR:
```
$ clang-tidy -p build mlir/lib/Conversion/MathToSPIRV/MathToSPIRV.cpp --checks=-*,misc-use-internal-linkage -fix -fix-errors
```
diff --git a/mlir/lib/Conversion/MathToSPIRV/MathToSPIRV.cpp b/mlir/lib/Conversion/MathToSPIRV/MathToSPIRV.cpp
index 1787e0a44f8f..50a93dec9446 100644
--- a/mlir/lib/Conversion/MathToSPIRV/MathToSPIRV.cpp
+++ b/mlir/lib/Conversion/MathToSPIRV/MathToSPIRV.cpp
@@ -485,8 +485,8 @@ struct RoundOpPattern final : public OpConversionPattern<math::RoundOp> {
//===----------------------------------------------------------------------===//
namespace mlir {
-void populateMathToSPIRVPatterns(const SPIRVTypeConverter &typeConverter,
- RewritePatternSet &patterns) {
+static void populateMathToSPIRVPatterns(const SPIRVTypeConverter &typeConverter,
+ RewritePatternSet &patterns) {
// Core patterns
patterns
.add<CopySignPattern,
```
Yields a build failure:
```
>>> referenced by MathToSPIRVPass.cpp
>>> MathToSPIRVPass.cpp.o:((anonymous namespace)::ConvertMathToSPIRVPass::runOnOperation()) in archive lib/libMLIRMathToSPIRV.a
clang++: error: linker command failed with exit code 1 (use -v to see invocation)
```
</pre>
<img width="1" height="1" alt="" src="http://email.email.llvm.org/o/eJysVVGPmzgQ_jXOy8iRYwMhDzywaSNVumqr3epO92jsCbhrbGSbdPPvTxCaZNs76dQWjRLAnm8-ZsbfyBhN6xArkj-Q_N1KjqnzofriXzBQHLpV4_W5-sukDpSVrqXJ6DM0o7EJZAI1hoAuQYdSg3Qa5DDYs3EtmATewcc_PjwRURM2W8EWYzXh2T0gHWZMDb01gfCDNQ3hh713JwzReEf44aNM3Wf__OnD059vn9ZqGIBS1aF6iUS8o4TXhO97ExUdI1LjEgYnLbXGvcgWgR7N6_xDMQQf4sLuB47aHI9AaWsSSMIPP0ut-XlfwmrjNL7CZltukcksO5bH9Tpncic0ql2WFbBhrMgywmpK6a_wnIvycLFf5EwyRjIGNCtzwvclEP5wvb0sxRRGleDJj04_Dp9kmkoER-OkBSJqGMbGGgWPwy3usomIfS9TNzWVqBd_It4D2T4QVgPhh8nEu4vR33Jd4Rb0KQ6rwcke4yAVzl27MKAnbzQMfhitTHiXm4V_JLxU3sUE89vP5wEv35gwAOFFun9B-H6ChCf8GkzCBeIZ07RzuALultiEP8Qkk1Hw-0lMXfE_aQAsZYC9DwjXDfPK26fpWkutidjv_XB-Nu21zpeo95Ix298GrY4gF704SmPHgDeN-c5HvL8YBDxiQKdQQ3OGNymJ8VvfXnf_y_raT0F4SXgpnXfn3o_x1gKE7y4tuaTtO4DLWhjdo3scMMg0H6Ry8uI7MA5kUJ05IVxOmzXNpJz3J0sSVs-CuRxSUcOsXtPNJGwYQPm-nzR4yglq-DqJNr6aBMprhA0QXo4RgZ4geYiIYNzJq4XL7sdsr3Ql9E7s5AqrzTbPt6XIeL7qqiIvG1Y0R4V5kXHRiJxtiqxQTLEtFxlfmYoznrOSbzZCbFm-FrhjyDJdMLktUG5JxrCXxq6tPfVrH9qViXHEapNn24KtrGzQxnkmcX4bE4TzaUaFavKizdhGkjFrYoo3nGSSxWp_Gy3GnaQ1GuLYthjnjxU1_Od0WI3BVl1Kw1yzuY1bk7qxWSvfT6Wxp29_dAj-C6pE-GFmHwk_LB9wqvg_AQAA__8cYkXB">