On Thu, Apr 22, 2021 at 5:51 PM Yonghong Song <yhs@xxxxxx> wrote: > > + > > +/* weak and shared between two files */ > > +const volatile int my_tid __weak = 0; > > +const volatile long syscall_id __weak = 0; > > Since the new compiler (llvm13) is recommended for this patch set. > We can simplify the above two definition with > int my_tid __weak; > long syscall_id __weak; > The same for the other file. > > But I am also okay with the current form > to *satisfy* llvm10 some people may still use. The test won't work with anything, but the latest llvm trunk, so " = 0" is useless. Let's remove it. Especially from the tests that rely on the latest llvm. No one can backport the latest llvm BPF backend to llvm10 front-end.
