[PATCH] D136452: [bazel] Add missing C++ style clang headers

Aaron Siddhartha Mondal via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 24 15:37:19 PDT 2022


aaronmondal added inline comments.


================
Comment at: utils/bazel/llvm-project-overlay/clang/BUILD.bazel:1478
+        "lib/Headers/**/*.h",
+        "lib/Headers/cuda_wrappers/algorithm",
+        "lib/Headers/cuda_wrappers/complex",
----------------
chapuni wrote:
> Seems odd to pass non-wildcard files to `glob`.
> 
> Instead, could we reorganize the pattern? Like,
> 
> * `lib/Headers/*.h`
> * `lib/Headers/cuda_wrappers`
> * `lib/Headers/openmp_wrappers`
> * `lib/Headers/ppc_wrappers`
> 
> (`clang/lib/Headers/hlsl` may be excluded here for now, since HLSL is not enabled in bazel build)
There would also be the possibility to use (note the last line)
```
"lib/Headers/*.h",
"lib/Headers/cuda_wrappers/*",
"lib/Headers/openmp_wrappers/*",
"lib/Headers/ppc_wrappers/*.h",
```
but I think just using the asterisks in this case should be clear enough. Omitting the asterisks won't work.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136452



More information about the llvm-commits mailing list