Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Thanks for doing this.
Don't we want to call them !toupper and !tolower, to match C++?
Thank you @zjaffal for working on this.
Please update also the docs at https://llvm.org/docs/TableGen/ProgRef.html
Francesco
| llvm/test/TableGen/string_ops.td | ||
|---|---|---|
| 33 | Unless there are compelling reasons for redirecting the test via !eq and !if, may I ask you to test the new operators directly? Something along the lines of: class ToLower<string str> {
string s = !tolower(str)
}
// CHECK-LABEL: def TEST1 {
// CHECK: string ="str";
// CHECK: }
def TEST1: ToLower<"sTr">;(something similar for !toupper) | |
Yep - my bad, I missed it. I should have taken another coffee before reviewing this patch. :)
Francesco