================
@@ -160,7 +160,7 @@ template<typename T> struct X;
// Make sure that the instantiated constructor initializes start and
// end properly.
-// CHECK-LABEL: define linkonce_odr void @_ZN1XIiEC2ERKS0_(ptr {{[^,]*}}
%this, ptr noundef nonnull align {{[0-9]+}} dereferenceable({{[0-9]+}}) %other)
unnamed_addr
+// CHECK-LABEL: define linkonce_odr void @_ZN1XIiEC2ERKS0_(ptr {{[^,]*}}
%this, ptr noundef nonnull align {{[0-9]+}} dereferenceable({{[0-9]+}}) %0)
unnamed_addr
----------------
mark-de-wever-sonarsource wrote:
It changes to the name used in the declaration. In this case the declaration in
the class is `X(const X &);` and the argument has no name. In this case it uses
`%0`. Would the declaration be `X(const X & x);` the name would change to `%x`.
I noticed only one test fails due to the change and it does not look that the
name of the argument is specifically part of the test. There seem to be no
other tests that are affected. So I'm not sure whether this specific change in
behaviour is wanted or important. I'd be happy to look into not changing it in
this case.
https://github.com/llvm/llvm-project/pull/142338
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
