[MERGE #5466 @meg-gupta] Fix dumping bytecode for intl/jsbuiltin

Meghana Gupta · Meghana Gupta · commit 88b1be47d29a · 2018-07-16T16:36:06.000-07:00
Merge pull request #5466 from meg-gupta:dumpfix Currently hits assert while printing source info, avoid going down the path for printing source info for builtins. Fixes OS#17198772
diff --git a/lib/Runtime/Base/FunctionBody.cpp b/lib/Runtime/Base/FunctionBody.cpp
@@ -4454,7 +4454,7 @@ namespace Js
 
     void FunctionBody::PrintStatementSourceLine(uint statementIndex)
     {
-        if (m_isWasmFunction)
+        if (m_isWasmFunction || this->GetUtf8SourceInfo()->GetIsLibraryCode())
         {
             // currently no source view support for wasm
             return;

Original file line number	Diff line number	Diff line change
`@@ -4454,7 +4454,7 @@ namespace Js`
`4454`	`4454`
`4455`	`4455`	`void FunctionBody::PrintStatementSourceLine(uint statementIndex)`
`4456`	`4456`	`{`
`4457`		`- if (m_isWasmFunction)`
	`4457`	`+ if (m_isWasmFunction \|\| this->GetUtf8SourceInfo()->GetIsLibraryCode())`
`4458`	`4458`	`{`
`4459`	`4459`	`// currently no source view support for wasm`
`4460`	`4460`	`return;`