[clang] [llvm] [ThinLTO][AIX] Add ThinLTO support for #pragma comment(copyright, ...) (PR #199398)

Wael Yehia via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 8 08:40:40 PDT 2026


w2yehia wrote:

Tony and I spoke on Friday, the fix looks good, one point to get a second opinion is the linkage of the comment string variable. 
To add to the [previous](https://github.com/llvm/llvm-project/pull/199398#issuecomment-4578593503) comment, in practice AIX seems to include filenames in the copyright string, so the de-duplication will not benefit them.

Also, if `internal` works for the thinLTO case (i.e. it doesn't prohibit functions from getting imported), it's always better to internalize than not. Tony was going to check if `internal` works. I think it should given that this similar test allows `foo` to be imported:
```
// foo.c
static int x;
int *foo() { return &x; }  // foo references the internal `x`
// main.c
int *foo();
int main() { return *foo(); }
```

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


More information about the cfe-commits mailing list