[PATCH] D140420: [Support] Update modulemap for TargetParser

Sam Elliott via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 20 13:31:57 PST 2022


lenary added inline comments.


================
Comment at: llvm/include/llvm/module.modulemap:403
+    // Exclude this; it is a forwarding header to TargetParser.
+    exclude header "ADT/Triple.h"
   }
----------------
steven_wu wrote:
> I am not sure if this exclude is necessary. It is ok for ADT to depend on TargetParser, correct?
> 
> We should either clean up all the include through ADT, or maybe just leave this. Same for all the TargetParser header in Support.
I got build issues until I added this. The `llvm/ADT/Triple.h` file now just forwards to `llvm/TargetParser/Triple.h`, so without this, you get a circular dependency of LLVM_Support -> TargetParser -> LLVM_Support


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140420



More information about the llvm-commits mailing list