[clang] [clang][NFC] Spell out DynTypedNode instead of auto (PR #114427)

2024-11-06 Thread Balazs Benics via cfe-commits

https://github.com/steakhal closed 
https://github.com/llvm/llvm-project/pull/114427
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits


[clang] [clang][NFC] Spell out DynTypedNode instead of auto (PR #114427)

2024-11-06 Thread Balazs Benics via cfe-commits

https://github.com/steakhal approved this pull request.

I don't really feel that this is relevant enough to be merged, but let's do it 
anyways.
If you find more things to refactor, or have plans for doing so, let me know.

https://github.com/llvm/llvm-project/pull/114427
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits


[clang] [clang][NFC] Spell out DynTypedNode instead of auto (PR #114427)

2024-11-02 Thread Chris Cotter via cfe-commits

ccotter wrote:

While working on #114244, I noticed the use of `auto` in 
`memberHasSameNameAsBoundNode`. The only other place I could find using `auto` 
with `Nodes.getNode` was in the second file I changed.

https://github.com/llvm/llvm-project/pull/114427
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits


[clang] [clang][NFC] Spell out DynTypedNode instead of auto (PR #114427)

2024-11-02 Thread Balazs Benics via cfe-commits

steakhal wrote:

Have you other similar cases in the Static Analyzer code base?

https://github.com/llvm/llvm-project/pull/114427
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits