Xiangling_L added a comment.

In D98265#2614245 <https://reviews.llvm.org/D98265#2614245>, @craig.topper 
wrote:

> Maybe this needs to be addressed in the forward declaration in 
> clang/include/clang/Basic/LLVM.h ?

Yeah, adding an additional llvm namespace is a certain another way to 
workaround this issue. But I think the fix we apply is also a quick way to 
workaround the build issue.  And as I mentioned, we would expect a real fix in 
a later patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98265

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to