[PATCH] D70183: Detect source location overflow due includes

Mikhail Maltsev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 19 06:02:11 PST 2019


miyuki added a comment.

In D70183#1751552 <https://reviews.llvm.org/D70183#1751552>, @dnsampaio wrote:

> Yes. It does return a non-valid FileID, and in builds without assert you get the expected error message.


I was asking about "It will still reach an false assert in builds that enable them". Has this been fixed?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70183





More information about the cfe-commits mailing list