[PATCH] D17005: Fix the LLVM_ENABLE_MODULES build after adding TargetOpcodes.def in r259726

David Kreitzer via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 8 15:12:59 PST 2016


DavidKreitzer created this revision.
DavidKreitzer added a reviewer: rsmith.
DavidKreitzer added a subscriber: llvm-commits.

Thanks, Richard, for the advice on how to fix to LLVM_ENABLE_MODULES build. I trust this is what you were suggesting?

I have been unable to test this, because I have not yet been able to get the stage 2 build working in my environment. (That is, the build fails before it hits the error reported by the buildbot.) The change is simple enough, though, that my plan is to just watch the buildbot and verify that it starts working again.


http://reviews.llvm.org/D17005

Files:
  include/llvm/module.modulemap

Index: include/llvm/module.modulemap
===================================================================
--- include/llvm/module.modulemap
+++ include/llvm/module.modulemap
@@ -37,6 +37,9 @@
   module Target {
     umbrella "Target"
     module * { export * }
+
+    // This is intended for (repeated) textual inclusion.
+    textual header "Target/TargetOpcodes.def"
   }
 
   // FIXME: Where should this go?


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D17005.47262.patch
Type: text/x-patch
Size: 410 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160208/bebb9c07/attachment.bin>


More information about the llvm-commits mailing list