DavidSpickett wrote:

I think the commit title would make more sense at a glance if it was saying 
what is being done instead of what's being fixed. How about:
```
[llvm][ARM] Emit MVE .arch_extension after .fpu directive if it does not 
include MVE features
```
Then the commit message is fine as is, and tells the reader why this was done 
and what it fixes.

https://github.com/llvm/llvm-project/pull/71545
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to