[clang] [clang][bytecode][NFC] Only collect non-null args if we have to (PR #152074)

[Shafik Yaghmour via cfe-commits]

================
@@ -2064,7 +2069,7 @@ bool Compiler<Emitter>::visitCallArgs(ArrayRef<const Expr *> Args,
         return false;
     }
 
-    if (FuncDecl && NonNullArgs[ArgIndex]) {
+    if (HasNonNullAttr && NonNullArgs[ArgIndex]) {
----------------
shafik wrote:

Couldn't we just use `!NonNullArgs.empty()` rather than have a flag boolean?

https://github.com/llvm/llvm-project/pull/152074