[PATCH] D14227: Add a new attribute: norecurse

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 2 11:15:05 PST 2015


joker.eph added a comment.

Patch looks good to me, but it would be nice to have more support for the attribute (or maybe at least wait a little bit for people to be able to object).


================
Comment at: include/llvm/IR/Function.h:328
@@ -327,1 +327,3 @@
 
+  /// @brief Determine if the function is known not to recurse, directly or
+  /// indirectly.
----------------
Just a nitpick: I believe we enabled autobrief and we shouldn't add it anymore when unnecessary.


Repository:
  rL LLVM

http://reviews.llvm.org/D14227





More information about the llvm-commits mailing list