steakhal added a comment.

In D95860#2538802 <https://reviews.llvm.org/D95860#2538802>, @balazske wrote:

> Existing test moved to Frontend, added new test.

Awesome, thanks!

On the functional side of things, I'm not sure if we should skip or not.
But simply skipping those seems like you are treating the symptom, not the 
cause.
Could you please give a rationale why can't we implement the subsequent logic 
in a way that it does not require `BeginFileID == EndFileID` to hold?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95860

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

Reply via email to