Skip to content

Commit c8c0762

Browse files
committed
Format getWarningFnName
1 parent 91bd6bd commit c8c0762

File tree

1 file changed

+24
-5
lines changed

1 file changed

+24
-5
lines changed

llvm/lib/Transforms/Instrumentation/MemorySanitizer.cpp

Lines changed: 24 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -820,12 +820,31 @@ MemorySanitizer::getOrInsertMsanMetadataFunction(Module &M, StringRef Name,
820820
StringRef getWarningFnName(bool TrackOrigins, bool Recover,
821821
bool EmbedFaultingInst) {
822822
StringRef warningFnName[2][2][2] = {
823-
{{"__msan_warning_noreturn", "__msan_warning_noreturn_instname"},
824-
{"__msan_warning", "__msan_warning_instname"}},
825823
{
826-
{"__msan_warning_with_origin_noreturn",
827-
"__msan_warning_with_origin_noreturn_instname"},
828-
{"__msan_warning_with_origin", "__msan_warning_with_origin_instname"},
824+
// TrackOrigins=false
825+
{
826+
// Recover=false
827+
"__msan_warning_noreturn", // EmbedFaultingInst=false
828+
"__msan_warning_noreturn_instname" // EmbedFaultingInst=true
829+
},
830+
{
831+
// Recover=true
832+
"__msan_warning", // EmbedFaultingInst=false
833+
"__msan_warning_instname" // EmbedFaultingInst=true
834+
},
835+
},
836+
{
837+
// TrackOrigins=true
838+
{
839+
// Recover=false
840+
"__msan_warning_with_origin_noreturn", // EmbedFaultingInst=false
841+
"__msan_warning_with_origin_noreturn_instname" // EmbedFaultingInst=true
842+
},
843+
{
844+
// Recover=true
845+
"__msan_warning_with_origin", // EmbedFaultingInst=false
846+
"__msan_warning_with_origin_instname" // EmbedFaultingInst=true
847+
},
829848
}};
830849

831850
return warningFnName[TrackOrigins][Recover][EmbedFaultingInst];

0 commit comments

Comments
 (0)