[PATCH] D102669: [analyzer][ctu] Fix wrong 'multiple definitions' errors caused by space characters in lookup names when parsing the ctu index file

Balázs Benics via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 21 01:00:43 PDT 2022


steakhal added a comment.

In D102669#3395364 <https://reviews.llvm.org/D102669#3395364>, @OikawaKirie wrote:

> IMO, maybe we can just leave a FIXME or something else in the test case and commit this patch to fix the original problem we want to fix.
> (of course, re-submit to rerun the test case on Linux)
> What do you think? @steakhal

I agree.
Please update the summary to have a reference to the discourse question you posted about this.


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

https://reviews.llvm.org/D102669



More information about the cfe-commits mailing list