[PATCH] D77971: [MC][X86][NFC] Disable branch align in non-text section

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 15 22:45:05 PDT 2020


MaskRay added inline comments.


================
Comment at: llvm/test/MC/X86/align-branch-section-type.s:3
+
+  # Check we only pad in a text section
+
----------------
Don't indent the file...

Use `## ` for comments


================
Comment at: llvm/test/MC/X86/align-branch-section-type.s:5
+
+  # CHECK:  Name: text
+  # CHECK:  AddressAlignment: 32
----------------
I have suggested the following in a previous comment:

```
# CHECK-LABEL: Name: text
# CHECK:       AddressAlignment: 32
```


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77971





More information about the llvm-commits mailing list