Yes as I don't have commit privileges.

In D61969#1527349, @jhenderson wrote:

In D61969#1519537, @xerofoify wrote:

Sorry about that I thought the patch name needed to change just not here. That's fixed and the error message is back to normal with the added condition and test case.

The error message I'm seeing still is wrong. I don't think you've updated the diff. Please do so.

Sorry about that I thought the patch name needed to change just not here. That's fixed and the error message is back to normal with the added condition and test case.

Sorry about the issues. I looked through the comments and it seems that the test cases below are correct and one needed to just be added by copying and pasting the other one but checking for the same addresses with a newer error message to check against. Hopefully this is correct now and thanks for the patience.

This is the updated patch. I wasn't sure if the test cases are correct, not sure if the syntax has issues. Otherwise this should be fixed now and thanks for the comments.

Yes you did and gnu generally errors here. My point was is there is a third case so the if should really be:
if (StartAddress >= StopAddress)

error("Start address should be less than stop address");

if (StartAddress < StopAddress)

error("Stop address should be greater than start address");

It fails after looking at the code again for the gnu binutils so it would be best to keep them compatible IMO. However merging my if statement together is better through so I
keep upload a new patch with the git diff comments as I was just using git format patch before. In addition it seems that there is also the case in binutils of lesser
than stop address so that should be added as well so I guess we really do need another if statement. Let me know if those changes work for you to keep compatibility with
gnu binutils.