ast added inline comments.
================ Comment at: clang/lib/CodeGen/CGExpr.cpp:3791 + if (hasBPFPreserveStaticOffset(Base)) + addr = wrapWithBPFPreserveStaticOffset(CGF, addr); + ---------------- eddyz87 wrote: > ast wrote: > > If I'm reading this correctly wrapping with preserve_static_offset doesn't > > prevent further preserver_access_index wrapping which is a wasted effort > > for pai at the end ? > Yes, pai calls are undone in `BPFPreserveStaticOffset.cpp:removePAICalls()`. > I can put back the logic that suppresses pai if preserve static offset is > present. I see. I guess I missed a previous discussion. Why this approach was chosen? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D133361/new/ https://reviews.llvm.org/D133361 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits