[llvm] [FileCheck]: Fix diagnostics for NOT prefixes (PR #78412)
Thomas Preud'homme via llvm-commits
llvm-commits at lists.llvm.org
Fri Jan 19 05:30:33 PST 2024
RoboTux wrote:
> @RoboTux I apologise, but I am a tad bit confused what you mean when you say that I would want to modify printMatch and printNotMatch to handle CheckNot.
>
> Do you mean I need to modify their definitions? I could simply pass in the correct prefix through reportMatchResult being called in CheckDag() and CheckNot(); Wouldn't that make it immaterial what Prefix is exactly held at CheckStr.Check(), if the current prefix is a DAG or NOT string?
>
> All other suggestions you gave I have already incorporated, and been able to get all tests to pass satisfactorily.
>
> Thanks a lot!
No my bad, it was me who missed something. It looks good now, thanks!
https://github.com/llvm/llvm-project/pull/78412
More information about the llvm-commits
mailing list