
<html>

<head>

<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">

<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>

</head>

<body dir="ltr">

<blockquote style="border-color: rgb(200, 200, 200); border-left: 3px solid rgb(200, 200, 200); padding-left: 1ex; margin-left: 0.8ex; color: rgb(102, 102, 102);" itemscope="" itemtype="https://schemas.microsoft.com/QuotedText">

<div>> Were these assertions firing on the existing test cases but on different architectures (if so, maybe it's worth adding versions of the test (or new RUN lines) that target those architectures so the failure's exercised on all buildbots with that target

 built?) - or were they only firing during something like a bootstrap? In which case maybe some more test coverage would be desirable to exercise these codepaths/conditions more directly?</div>

</blockquote>

<div>I think adding more test cases here to cover that code path is probably appropriate; the asserts were specifically in the DWARF emission step of the pipeline, asserting that only non-constant arguments were used for DBG_VALUE_LISTs. This assertion is not

 strictly correct (it was used mainly for testing) but it didn't trigger at all on a number of large projects, so slipped by relatively unnoticed. The fact that emitting constant operands for DBG_VALUE_LIST is not covered by any kind of test is an issue that

 should be fixed, along with any other currently untested operand types.<br>

</div>

</body>

</html>