================
@@ -11576,22 +11584,6 @@ static bool CheckTargetCausesMultiVersioning(Sema &S,
FunctionDecl *OldFD,
}
}
- if (NewTVA) {
----------------
labrinea wrote:
This is checking things like
```
//expected-note@+1 {{previous declaration is here}}
void __attribute__((target_version("bti+flagm2"))) one(void) {}
//expected-error@+1 {{multiversioned function redeclarations require identical
target attributes}}
void __attribute__((target_version("flagm2+bti"))) one(void) {}
```
(example taken from clang/test/Sema/attr-target-version.c)
At this point we know that the new declaration is triggering multiversioning
therefore the old was the (implicit or explicit) default.
https://github.com/llvm/llvm-project/pull/96628
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits