This is an archive of the discontinued LLVM Phabricator instance.

[lldb] Add nodebug attribute to import-std-module/sysroot test
ClosedPublic

Authored by teemperor on Oct 11 2019, 1:09 AM.

Details

Summary

So far we rely on the default argument and the fact that we don't call this
inline function in our actual main.cpp to make sure that this function can only
be called if LLDB loads this header as a C++ module. This patch just adds
the nodebug attribute as yet another measure to make sure LLDB can't call this
function without the standard module loaded. Note that the test is already
requiring clang for the sysroot setup, so its fine that this is a Clang specific attribute.

Diff Detail

Event Timeline

teemperor created this revision.Oct 11 2019, 1:09 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 11 2019, 1:09 AM
This revision was not accepted when it landed; it landed in state Needs Review.Oct 23 2019, 9:37 AM
This revision was automatically updated to reflect the committed changes.