[PATCH] D136134: [NFC] [DirectX backend] move ResourceClass into llvm.

[Chris Bieneman via Phabricator via cfe-commits]

beanz added inline comments.


================
Comment at: clang/lib/Sema/HLSLExternalSemaSource.cpp:20
 #include "clang/Sema/Sema.h"
+#include "llvm/Frontend/HLSL/HLSLResource.h"
 
----------------
python3kgae wrote:
> python3kgae wrote:
> > beanz wrote:
> > > You need to add FrontendHLSL to the Sema/CMakeLists.txt file too.
> > Good catch.
> > Not sure why both local build and the pre-commit check cannot hit it. :(
> I think the reason it works is that it only used the enum decl in the header, not anything which needs to link.
> Do we need to add FrontendHLSL to CMakeLists in this case?
The first header include added to a component should add the linkage dependency (even if it isn't strictly needed).

It is too easy to add a linkage dependency later without realizing it isn't already specified. Because of how static archive linking works you don't necessarily notice the missing dependency because many of our tools link both Sema and CodeGen where there is already a dependency.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136134