[PATCH] D108421: Mark openmp internal global dso_local
Alexey Bataev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Aug 24 04:14:28 PDT 2021
ABataev added a comment.
In D108421#2962151 <https://reviews.llvm.org/D108421#2962151>, @lebedev.ri wrote:
> In D108421#2961611 <https://reviews.llvm.org/D108421#2961611>, @kamleshbhalui wrote:
>
>> updated test and make changes local to auto generated global vars for lock.
>
> I do not understand why the most original diff here was wrong.
> Is is the wrong default for `getOrCreateInternalVariable()` to default to `dso_local`?
Need to check that the linkage/visibility of the created vars is not changing in place. If it is not changing, then it is good to go. Otherwise, need to add some another function/extra parameter with linkage/visibility settings.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D108421/new/
https://reviews.llvm.org/D108421
More information about the cfe-commits
mailing list