================
@@ -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
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to