[llvm] [CFI][annotation] Leave alone function pointers in function annotations (PR #80173)

[Nikita Popov via llvm-commits]

================
@@ -1837,6 +1851,16 @@ LowerTypeTestsModule::LowerTypeTestsModule(
   }
   OS = TargetTriple.getOS();
   ObjectFormat = TargetTriple.getObjectFormat();
+
+  // Function annotation describes or applies to function itself, and
+  // shouldn't be associated with jump table thunk generated for CFI.
+  GlobalAnnotation = M.getGlobalVariable("llvm.global.annotations");
+  auto *C = dyn_cast_or_null<Constant>(GlobalAnnotation);
+  if (C && C->getNumOperands() == 1) {
+    C = cast<Constant>(C->getOperand(0));
+    for (auto &Op : C->operands())
----------------
nikic wrote:

```suggestion
    for (Value *Op : C->operands())
```
If you're not using `Use &` anyway.

https://github.com/llvm/llvm-project/pull/80173