Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Unit Tests
Time | Test | |
---|---|---|
420 ms | linux > HWAddressSanitizer-x86_64.TestCases::sizes.cpp |
Event Timeline
This conversion LGTM.
It occurred to me that I don't know if we have tests that actually cover the conversion that we're removing here, namely LLVM IR -> assembly. e.g. after a really quick search I didn't see an ll test testing extern_weak globals in llvm/test/CodeGen/WebAssembly/ or llvm/test/MC/WebAssembly/. Did I just miss it?
Good question. I do normally check when I do these conversions but maybe I forgot this time. I will double check before I land.
llvm/test/CodeGen/WebAssembly/weak.ll | ||
---|---|---|
18 | nit: extra space before CHECK |
llvm/test/CodeGen/WebAssembly/weak.ll | ||
---|---|---|
18 | I've been using this technique in the lld tests to that the : chars line up and allow expectations to be more easily read IMHO. But this is a different place so I will change. |
nit: extra space before CHECK