arsenm added a comment.

I think getting too specific as to why in the messages may confuse people. 
Especially "might not update memory" is not super helpful. It sounds like the 
instruction is just entirely unreliable



================
Comment at: llvm/lib/Target/AMDGPU/SIISelLowering.cpp:12212
+            "A floating-point atomic instruction will generate an unsafe"
+            " hardware instruction which may not follow denorm mode");
+      }
----------------
Should not abbreviate denormal


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106891

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to