[clang] [llvm] [PowerPC][AIX] Support #pragma comment copyright for AIX (PR #178184)
Hubert Tong via cfe-commits
cfe-commits at lists.llvm.org
Sat May 9 09:54:40 PDT 2026
================
@@ -2079,6 +2084,13 @@ class CodeGenModule : public CodeGenTypeCache {
/// false, the definition can be emitted lazily if it's used.
bool MustBeEmitted(const ValueDecl *D);
+ /// Emit load-time comment metadata for #pragma comment(copyright, ...).
+ /// This adds the copyright string to the module's named metadata, which will
+ /// be processed by the IR passes to embed it in the .text section
+ /// of the generated object file, where it will be loaded into memory at
+ /// program runtime.
+ void EmitLoadTimeComment();
----------------
hubert-reinterpretcast wrote:
- It is very odd to place this between `MustBeEmitted` and `MayBeEmittedEagerly`. Please add to the end of the class.
- To match the naming of this function (and the metadata, etc.), the comment should be less `#pragma comment(copyright, ...)`-centric.
- In the front-end code, let's just say what the front-end does and leave the responsibility of the other components to said components. In particular, that leaves us with less to fix if things change.
```suggestion
/// Emit the load-time comment metadata (e.g., from
/// #pragma comment(copyright, ...)) for the translation unit.
void EmitLoadTimeComment();
```
https://github.com/llvm/llvm-project/pull/178184
More information about the cfe-commits
mailing list