jrtc27 added inline comments.
================ Comment at: clang/lib/Driver/SanitizerArgs.cpp:573 + Kinds & SanitizerKind::ShadowCallStack) + << "-msmall-data-limit=0"; + } ---------------- jrtc27 wrote: > paulkirth wrote: > > jrtc27 wrote: > > > Why is this an error? It may be a misguided thing to enable but it is > > > 100% supported to combine this. All the limit does is put things in > > > .sdata, but they can still be addressed just fine. > > My understanding is that (whether they should or not) linkers are using the > > presence of the `.sdata` section to enable gp relaxation. > That is not true Putting variables in .sdata merely makes it more likely that the linker will be able to relax accesses to them to use GP. Variables outside .sdata still can and will be relaxed if within range of whatever gets picked for `__global_pointer$`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146463/new/ https://reviews.llvm.org/D146463 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
