[PATCH] D61947: Merge of Global Constants not happening on Aarch64

[Tim Northover via Phabricator via llvm-commits]

t.p.northover added inline comments.


================
Comment at: test/CodeGen/AArch64/global_merge_aarc64_ac6.ll:25-35
+attributes #0 = { minsize norecurse nounwind optsize readonly "correctly-rounded-divide-sqrt-fp-math"="false" "disable-tail-calls"="false" "less-precise-fpmad"="false" "min-legal-vector-width"="0" "no-frame-pointer-elim"="true" "no-frame-pointer-elim-non-leaf" "no-infs-fp-math"="false" "no-jump-tables"="false" "no-nans-fp-math"="false" "no-signed-zeros-fp-math"="false" "no-trapping-math"="false" "stack-protector-buffer-size"="8" "target-cpu"="cortex-a53" "target-features"="+aes,+crc,+crypto,+fp-armv8,+neon,+sha2" "unsafe-fp-math"="false" "use-soft-float"="false" }
+
+!llvm.module.flags = !{!0}
+!llvm.ident = !{!1}
+
+!0 = !{i32 1, !"wchar_size", i32 4}
+!1 = !{!"clang version 9.0.0 (https://git.llvm.org/git/clang.git/ 7a6d690bb52df60e979a7c7f9d03a774009bb443) (https://git.llvm.org/git/llvm.git/ 71abe189dddb95b0d5782e0866058b9a92ff2402)"}
----------------
There's no need to include most of this extra stuff in the test. You might need to replace the `#0` on the definition of `@func` with `minsize` though.


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

https://reviews.llvm.org/D61947