[llvm] [DirectX] Infrastructure to collect shader flags for each function (PR #112967)

S. Bharadwaj Yadavalli via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 28 09:37:49 PDT 2024


bharadwajy wrote:

> I stopped reading this after seeing the string construction in the comparison function. I'm either missing something big about this change, which means I'd need to review again once I've been corrected, or this PR is going to be somewhat different anyway, so I'll wait until that's resolved before looking at it again.

Lexicographic sort of functions by their signatures seemed the appropriate. I looked at using [`FunctionComparator::compareSignature()`](https://github.com/llvm/llvm-project/blob/7a710110fcb2ad5d903ec41ba6a63193cf03edc0/llvm/lib/Transforms/Utils/FunctionComparator.cpp#L946) but did not see its usage in the sources other than in tests, so wasn't sure if its usage is encouraged or not.

What are the concerns?

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


More information about the llvm-commits mailing list