<table border="1" cellspacing="0" cellpadding="8">
    <tr>
        <th>Issue</th>
        <td>
            <a href=https://github.com/llvm/llvm-project/issues/54237>54237</a>
        </td>
    </tr>

    <tr>
        <th>Summary</th>
        <td>
            [MLIR] `OpAsmParser::parseRegion` has redundant argLocations array?
        </td>
    </tr>

    <tr>
      <th>Labels</th>
      <td>
            mlir:core
      </td>
    </tr>

    <tr>
      <th>Assignees</th>
      <td>
      </td>
    </tr>

    <tr>
      <th>Reporter</th>
      <td>
          lattner
      </td>
    </tr>
</table>

<pre>
    The parseRegion signature looks like this:

```
 virtual ParseResult parseRegion(Region &region,
                                  ArrayRef<OperandType> arguments = {},
                                  ArrayRef<Type> argTypes = {},
                                  ArrayRef<Location> argLocations = {},
                                  bool enableNameShadowing = false)
```

However, `OperandType` in the arguments list already carries a SMLoc for the operand location. I think we can drop the `argLocations` array.  Is there any obvious problem with doing that?

-Chris
</pre>
<img width="1px" height="1px" alt="" src="http://email.email.llvm.org/o/eJylU1Fv2yAQ_jX45dSIQmLXD35Ik1WL1G5T2z9wDmebBZsIcKL8-4GTap40aVJnIeB88N333R21VZfqvSM4ovP0Sq22A3jdDhhGR2CsPXgw-kAQOu2ZXDO-ZfxjzvltTCactAsjGvhxxfKjCXNcJh5uAZjI3e3X5nb3n9_aOby8UsPk5vuRHA7q_XIkJr8AunbsaQgemNwCKx5Zsf0c8Awxbf8f8NnuMSSdV9AP85PAtbUGaMDa0Dfs6a1DZc96aCe0Bo0nJsq_luY6f7VnOpGLESF65lnMOeghlphmyTTaB0DjCNUF9uicjglBeHuJKqCxbjpuryCxUa7KFrBLnTIc4Ezx0gDK2eN0MsaYZyCFxJSoBcDOpxOx3XC4gK1P2o4ejs5GnT2cdehA2SQzdBiYfJpLutt0TvtMVVKVssQs6GCoYqvHl-fdK1ttr0LXvp960qUGlut5T0YaHXpwpMZB4RD-LNPEMIbMRmeqLoTj9ATEUxxt5DXWi73to2HM6WO5i8R_0j5EU3s_ko-b1VLIIuuqsuS5Wgl6EA0vl0Wd1_lDsVzJGiVyrCkzcTI-8WdC9EYnwnvrYl1FFJPpSnAhuOQF5zxiLZp7ul_KoimlUKjyhi059ajNIjFZWNdmrppI1WProzPV1P92ok9PnaaEJXwcQ2fjDQxhIJdN_KuJ_C-Cnz6a">