StephenFan added inline comments.
================ Comment at: clang/test/CodeGen/lifetime2.c:42 + // O2: @llvm.lifetime.start.p0i8(i64 1 bar(&x, 1); + // O2: @llvm.lifetime.end.p0i8(i64 1 ---------------- StephenFan wrote: > vitalybuka wrote: > > It assume this will break Msan > > Transforms/Instrumentation/MemorySanitizer.cpp:1298 as it assume variable > > is not initialized on start > > > > ``` > > void goto_bypass(void) { > > { > > char x; > > l1: > > bar(&x, 1); > > if (x) > > goto l1 > > } > > goto l1; > > } > > ``` > Yes. I still need some time to see how to deal with it. Candidate solution: D129991 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129448/new/ https://reviews.llvm.org/D129448 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits