[PATCH] D142196: [clang][Lex] Add back PPCallbacks::FileNotFound

Jonas Hahnfeld via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 20 04:54:00 PST 2023


Hahnfeld added a comment.

I just checked and this "simplified" version of the callback would give us enough information to implement all we need in ROOT (plus some, the `bool` return value is nicer than temporarily playing with `SuppressIncludeNotFoundError`). For an upstream test, I could implement a unit test in `clang/unittests/Lex/PPCallbacksTest.cpp` or a "toy" tooling where `#include <file.h?>` is ignored if `file.h` doesn't exist? What do you think (assuming the general idea of adding this callback is fine)?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142196



More information about the cfe-commits mailing list