[PATCH] D62424: Tests for nosync function attribute.

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added inline comments.


================
Comment at: llvm/test/Transforms/FunctionAttrs/nosync.ll:70
+
+; CHECK: define i32 @load_acquire(i32* nocapture readonly) norecurse nounwind uwtable
+define i32 @bar(i32* nocapture readonly) norecurse nounwind uwtable {
----------------
Add a
`CHECK-NOT: nosync`
after the check line because you could otherwise have `nosync` at the end and pass anyway.



================
Comment at: llvm/test/Transforms/FunctionAttrs/nosync.ll:116
+
+; CHECK: define void @call_nosync_function() noinline nounwind readnone uwtable nosync
+define void @call_nosync_function() nounwind uwtable noinline {
----------------
FIXME: nosync missing
instead of nosync in the attribute list

We should be able to pass the test right now so we can integrate it already.
To this end, please also use the string attribute syntax `"nosync"` which should be accepted right now. (You can run lit on the test)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62424