[PATCH] D90275: [clang][IR] Add support for leaf attribute
    Aaron Ballman via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Tue Dec  8 06:06:19 PST 2020
    
    
  
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
Aside from a testing request, the frontend parts LGTM. I don't know enough about the LLVM side to feel comfortable signing off on it. @jdoerfert, would you mind approving that part?
================
Comment at: clang/test/Sema/attr-leaf.c:4
+
+void f() __attribute__((leaf));
----------------
Can you also add tests that show the attribute diagnoses being written on the wrong subject or when given arguments? Also, can you add a test showing the attribute on a function definition with a FIXME comment about wanting to diagnose that case?
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D90275/new/
https://reviews.llvm.org/D90275
    
    
More information about the llvm-commits
mailing list