erichkeane added inline comments.
================
Comment at: clang/lib/Sema/SemaDeclAttr.cpp:8794
+ case ParsedAttr::AT_Pure:
+ handleSimpleAttribute<PureAttr>(S, D, AL);
+ if (!D->hasAttr<NoThrowAttr>()) {
----------------
First, I'm not a fan of doing this in this list ast all, we shouldn't really
have logic here, this is mostly just a jump table.
Second, I guess I'm not getting why IRGen cannot figure out from the AST that
Pure/Const/NoAlias all imply nothrow?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D137901/new/
https://reviews.llvm.org/D137901
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
