curdeius added a comment.

In D93938#2612952 <https://reviews.llvm.org/D93938#2612952>, 
@HazardyKnusperkeks wrote:

> In my opinion you should then, either temporarily deactivate the test, or fix 
> the bug first. A failing test blocks the pipeline and confuses everyone 
> working on the project.

+1

I got confused about this. I know that there was some discussion about this 
failing test but I thought that the plan was to fix it (as it should). Also, 
that's what one expects in a revision called "Fixed AfterEnum handling" :).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93938

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to