[PATCH] D71037: [Diagnostic] Add ftabstop to -Wmisleading-indentation

Tyker via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 3 16:47:05 PST 2020


Tyker closed this revision.
Tyker added a comment.

In D71037#1803704 <https://reviews.llvm.org/D71037#1803704>, @mstorsjo wrote:

> In D71037#1803574 <https://reviews.llvm.org/D71037#1803574>, @Tyker wrote:
>
> > In D71037#1803361 <https://reviews.llvm.org/D71037#1803361>, @xbolva00 wrote:
> >
> > > Can you add a test case for that crash? Otherwise OK.
> >
> >
> > as i said. the bug can only occur for one line files. so we can't have a run line and a the crashing line. so i wasn't able to make a test for it.
>
>
> Is it possible to have the `RUN` bit at the end of the line in a comment, after the test code itself?


it is possible.

i committed the fix and the tests.


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

https://reviews.llvm.org/D71037





More information about the cfe-commits mailing list