[PATCH] D148601: [Clang] Handle Error message to output proper Prefix

Usman Akinyemi via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 19 00:52:14 PDT 2023


Unique_Usman added a comment.

In D148601#4279334 <https://reviews.llvm.org/D148601#4279334>, @tbaeder wrote:

> I am not 100% sure about the semantics of passing multiple prefixes, i.e. if the error is emitted for all prefixes individually or if it's only emitted if no `expected` line for any of the prefixes is found. In the latter case we should probably add all the prefixes to the error message.

I tested different scenerios e.g added more than one RUN lines with different value of -verify, what I concluded on is that if we have multiple  RUN lines with each of them having no directive, the prefixes generated is always of the first occurence  with no  expected directive so, the error is always generated for the first occurence with no expected directive.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148601



More information about the cfe-commits mailing list