[PATCH] D58102: Support X86 Control-flow Enforcement Technology (CET) in LLD

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added inline comments.


================
Comment at: test/ELF/i386-feature-1-and.s:23
+# RUN: llvm-readelf -n %t | FileCheck --check-prefix=NGP2 %s
+# RUN: llvm-readelf -S %t | FileCheck --check-prefix=SEC2 %s
+
----------------
These `llvm-readelf -S %t` RUN commands can share the same check-prefix. They just check if `.note.gnu.property` is present. (Actually I'm thinking if they can be deleted if the related `llvm-readelf -n` is not empty)


================
Comment at: test/ELF/i386-feature-1-and.s:39
+
+# Check .note.gnu.protery
+# SEC1: .note.gnu.property
----------------
Typo. The comment may be less useful. A brief comment above the `llvm-mc` `ld.lld` commands explaining what the test is intended for may be more helpful to readers.


================
Comment at: test/ELF/x86-64-feature-1-and.s:39
+
+# Check .note.gnu.protery
+# SEC1: .note.gnu.property
----------------
Typo. `protery -> property`


Repository:
  rLLD LLVM Linker

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

https://reviews.llvm.org/D58102