[PATCH] D86974: [IRSim] Adding basic implementation of llvm-sim.

Markus Böck via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 22 10:18:10 PDT 2021


zero9178 added a comment.

In D86974#2641788 <https://reviews.llvm.org/D86974#2641788>, @AndrewLitteken wrote:

> There's a bug with one of the Windows buildbots where the error message doesn't match (even though it looks like it does), but I haven't quite been able to figure out why it's failing yet.  I think it has something do to with the fact that it's reporting an error, and that there is a FileCheck prefix.
>
> https://lab.llvm.org/buildbot#builders/123/builds/3498

Visual Studios Standard library starts their errors in lower case, which do not match the hardcoded values you used in your tests. Take a look at the lit feature added here https://reviews.llvm.org/rGc52fe0b02172e707aa2ba38cd2e01a1fc70dd0da, you should be able to use it to expand to the correct platform specific error message


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D86974/new/

https://reviews.llvm.org/D86974



More information about the llvm-commits mailing list