[PATCH] D129709: [clang][CodeGen] add fn_ret_thunk_extern to synthetic fns

[Fangrui Song via Phabricator via cfe-commits]

MaskRay added inline comments.


================
Comment at: llvm/lib/IR/Function.cpp:357
   }
+  if (M->getModuleFlag("fn_return_thunk_extern"))
+    B.addAttribute(Attribute::FnRetThunkExtern);
----------------
How about "function_return_thunk_extern"? fn doesn't abbreviate much. Having 
the `function_return` substring makes it easy for grepping.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129709

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