[PATCH] D148776: [Modules] Move modulemaps to header search directories. NFC intended.

David Blaikie via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 8 18:44:23 PDT 2023


dblaikie added a comment.

Got a link to a design discussion motivating this change?

I'd have thought it made sense to put modulemaps in subdirectories - since they cover the whole directory, putting them in the root of an include path would be problematic if there are multiple distinct projects in that same path. And I didn't think this involved searching through subdirectories, but walking up parent directories from the included file...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148776



More information about the cfe-commits mailing list