[clang] [Clang][AArch64] Remove unwarranted 'cannot be used in non-streaming mode' diagnostic. (PR #150592)
Benjamin Maxwell via cfe-commits
cfe-commits at lists.llvm.org
Fri Jul 25 05:37:14 PDT 2025
================
@@ -5988,11 +5988,9 @@ bool clang::IsArmStreamingFunction(const FunctionDecl *FD,
if (FD->hasAttr<ArmLocallyStreamingAttr>())
return true;
- if (const Type *Ty = FD->getType().getTypePtrOrNull())
----------------
MacDue wrote:
> If for example, I were to call FD->dump() in this function, then dump will run into an assertion that the type is null.
Any assumptions about like this should be backed by an explicit assert, dereferencing a null-pointer without an assert can lead to less obvious errors/issues (and it's not clear what the authors intent was).
https://github.com/llvm/llvm-project/pull/150592
More information about the cfe-commits
mailing list