[PATCH] D108421: Mark openmp internal global dso_local

Alexey Bataev via Phabricator via llvm-commits llvm-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 llvm-commits mailing list