[PATCH] D130553: [clang][lld][cmake] Simplify header dirs

John Ericson via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 4 05:17:44 PDT 2022


Ericson2314 added a comment.

@tstellar There wasn't a configuration necessitating these changes.

The CMake config file currently distinguishes a main include dir and secondary include dir, which feels to me like it is leaking the implementation details of which headers are generated or not. I was hoping by replacing the instances where those are used with the list one, we'd eventually be able to deprecate the two underlying variables and just have the list one left.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130553



More information about the cfe-commits mailing list