| Issue |
107723
|
| Summary |
Consolidate `StrCmpOptionName` duplicate functions in OptEmitter.cpp and OptTable.cpp
|
| Labels |
new issue
|
| Assignees |
|
| Reporter |
jurahul
|
Both files contain function to do a *almost* case-insensitive lexicographic comparison, but there is one subtle difference:
1. The one in OptTable.cpp considers 2 strings of same length identical if they are identical case-insensitive-ly. The one in OptEmitter.cpp considers such strings as order per their case-sensitive comparison (with a call to strcmp(A,B))
Performance/code wise, the one in OptTable.cpp seems better as it uses StringRef and its API to do most of the work.
This bug is to consolidate the 2 functions to be identical as well as share the functions as well.
![]()
_______________________________________________
llvm-bugs mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs
