This is an archive of the discontinued LLVM Phabricator instance.

msan, codegen, instcombine: Keep more lifetime markers used for msan
ClosedPublic

Authored by vitalybuka on Aug 23 2019, 6:50 PM.

Event Timeline

vitalybuka created this revision.Aug 23 2019, 6:50 PM
Herald added projects: Restricted Project, Restricted Project, Restricted Project. · View Herald TranscriptAug 23 2019, 6:50 PM
Herald added subscribers: llvm-commits, Restricted Project, cfe-commits, hiraditya. · View Herald Transcript

return hwasan

fix compilation error

Harbormaster completed remote builds in B37235: Diff 216992.

update comment

lebedev.ri added inline comments.
llvm/test/Transforms/InstCombine/lifetime-sanitizer.ll
37–50

Would be good to have end-to-end test (-O0/-O1/-O2/-O3) in PhaseOrdering.

eugenis accepted this revision.Aug 26 2019, 11:01 AM

LGTM

llvm/test/Transforms/InstCombine/lifetime-sanitizer.ll
37–50

Do you mean exact same test, but with -O0/../-O3 instead of -instcombine?

This revision is now accepted and ready to land.Aug 26 2019, 11:01 AM
lebedev.ri added inline comments.Aug 26 2019, 11:04 AM
llvm/test/Transforms/InstCombine/lifetime-sanitizer.ll
37–50

Yes, but please put it into PhaseOrdering directory,

This revision was automatically updated to reflect the committed changes.