[PATCH] D126929: Add sanitizer metadata attributes to clang IR gen.

Kirill Stoimenov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 3 11:08:24 PDT 2022


kstoimenov added inline comments.


================
Comment at: clang/lib/CodeGen/SanitizerMetadata.cpp:48-50
+  GVSanitizerMetadata Meta;
+  if (GV->hasSanitizerMetadata())
+    Meta = GV->getSanitizerMetadata();
----------------
Would it make sense to create two separate functions something like reportGlobalNew and reportGlobalLegacy to make it more clear which one is which? Then you code will be something like the one below? You can come up with better names for functions, I am sure. 

```
reportGlobal(...) {
  reportGlobalNew(...);
  reportGlobalLegacy(...);
}
```



Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D126929/new/

https://reviews.llvm.org/D126929



More information about the cfe-commits mailing list