<div dir="ltr">It's ensuring that macros like `define N 1024` don't create mapping regions when the integer literal<div>is implicitly converted. It also makes sure that literals in ternary conditional create mapping regions</div>
<div>- e.g. there will be a mapping region for 1 and 0 in `cond? 1 : 0` .</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-08-20 23:49 GMT-07:00 Justin Bogner <span dir="ltr"><<a href="mailto:mail@justinbogner.com" target="_blank">mail@justinbogner.com</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">Alex L <<a href="mailto:arphaman@gmail.com">arphaman@gmail.com</a>> writes:<br>

> These two patches remove the mapping for implicit cast expression and<br>
> modify the "ignore if not extended" flag to be "ignore if file only<br>
> has this region". I'm sending them out together as they both partially<br>
> influence a common testcase.<br>
<br>
</div></div>Why? What is this accomplishing? As far as I can tell one of these<br>
patches removes a CHECK line and the other adds back the exact same<br>
check. I guess this changes how that particular construct is modelled?<br>
If so, it might be better to just make this one patch so it's clear that<br>
you're not removing functionality, but changing how it's done.<br>
</blockquote></div><br></div>