Fix module map to create a module for the configured header Config/abi-breaking.h

[mehdi_amini]

Summary

A client of a header that relies on ABI breaking should get the macro
exported there.
Before this, the unittest for Support/Error including Support/Error.h
didn't get the macro exported by the Support module, because the
latter only re-export its submodules and included module, not
textual headers.

Hopefully, it'll also fix the build with local submodule visibility,
since the LLVM_Utils contains two submodules: ADT and Support. They
both include abi-breaking.h that defines a symbol. The textual
inclusion lead to a double definition of the symbol which broke
the parent module.

[bruno]

LGTM! I would wait on @rsmith input as well.

llvm/include/llvm/module.modulemap.build

6

LLVM_Config_gen => LLVM_Config_ABI_Breaking

[mehdi_amini]

llvm/include/llvm/module.modulemap.build

6

Is used LLVM_Config_gen for generated files, this is not the only header in the build directory and I was wondering if we shouldn't export more? (Not urgent though)

[mehdi_amini]

Rename the module as suggested by Bruno

[mehdi_amini]

I'm pushing this and @rsmith can comment post-commit.

[mehdi_amini]

This fixed the build for this bot: http://lab.llvm.org:8011/builders/clang-x86_64-linux-selfhost-modules/builds/1177

[bruno]

Thanks Mehdi

llvm/include/llvm/module.modulemap.build

 // This is copied into the build area for a $src != $build compilation.
 module LLVM_Support_DataTypes {
   header "Support/DataTypes.h"
   export *
 }
+module LLVM_Config_ABI_Breaking {
+  header "Config/abi-breaking.h"
+  export *
+}