[PATCH] D125974: [clang] Limit bitcode option ignorelist to Darwin

Anders Waldenborg via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 24 02:42:53 PDT 2022


wanders added a comment.

This looks sensible to me. But I'm not an expert in the area (and still not 100% clear on the purpose of the ignorelist) so can't really say if the added comments captures the intent accurately.



================
Comment at: clang/test/Driver/embed-bitcode-elf.c:1
+// RUN: %clang -c -target aarch64-linux-android21 %s -fembed-bitcode -o %t.o \
+// RUN: -ffunction-sections -fdata-sections -fstack-protector-strong
----------------
Should this test have a `// REQUIRES: aarch64-registered-target`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125974



More information about the cfe-commits mailing list