[PATCH] D100689: [funcattrs] Consistently check call site attributes

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 19 09:04:05 PDT 2021


reames added inline comments.


================
Comment at: llvm/test/Transforms/FunctionAttrs/willreturn-callsites.ll:1
-; RUN: opt -function-attrs -S %s | FileCheck %s
+; RUN: opt -inferattrs -function-attrs -S %s | FileCheck %s
 
----------------
nikic wrote:
> Why is the inferattrs here needed now?
Because the declarations are only partially annotated, and func-attrs which is an CGSCC pass does not visit declarations, only inferattrs does.  


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100689



More information about the llvm-commits mailing list