[PATCH] D123498: [clang] Adding Platform/Architecture Specific Resource Header Installation Targets

[Qiongsi Wu via Phabricator via cfe-commits]

qiongsiwu1 added a comment.

In D123498#3458967 <https://reviews.llvm.org/D123498#3458967>, @pengfei wrote:

>> Maybe I can ask cmake to check for architecture/targets during configuration and select the headers automatically, but that is beyond the scope of this patch.
>
> I'm not familar with cmake, but I guess it might be doable. I once verified the X86 headers by command `echo '#include <x86intrin.h>' | clang -x c -E - | grep '#.*clang.*h' | grep -o '[^\/]*\.h' |sort|uniq`.
> Notice, targets like X86 doesn't have a single entry. But it still available with a bit more work.

Ah I see! Let me look into adopting something along this line. Maybe eventually we can remove all the lists (or reduce the lists to minimal) here.

Thanks for the input!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123498